#14303: "sage -i" should only run sage-location if it exists
---------------------------------+------------------------------------------
       Reporter:  jhpalmieri     |         Owner:  GeorgSWeber 
           Type:  enhancement    |        Status:  needs_review
       Priority:  minor          |     Milestone:  sage-5.9    
      Component:  build          |    Resolution:              
       Keywords:  sage-location  |   Work issues:              
Report Upstream:  N/A            |     Reviewers:              
        Authors:  John Palmieri  |     Merged in:              
   Dependencies:                 |      Stopgaps:              
---------------------------------+------------------------------------------

Old description:

> During Sage installation, I sometimes see an spkg fail to build, for
> example [#13947 zn_poly]. Then I run
> {{{
> $ ./sage -i spkg/standard/zn_poly-0.9.p10.spkg
> }}}
> If this installs successfully, it still has a return code of 127 (on OS
> X, at least), because the installation runs `sage-location` at the end,
> which hasn't been installed yet. So perhaps `sage -i ...` should only run
> `sage-location` if it already exists.  This way something like
> {{{
> $ make || ./sage -i spkg/standard/zn_poly-0.9.p10.spkg && make ptestlong
> }}}
>
> might work the way I want, and building Sage would require less
> babysitting.

New description:

 During Sage installation, I sometimes see an spkg fail to build, for
 example [http://trac.sagemath.org/sage_trac/ticket/13947 zn_poly]. Then I
 run
 {{{
 $ ./sage -i spkg/standard/zn_poly-0.9.p10.spkg
 }}}
 If this installs successfully, it still has a return code of 127 (on OS X,
 at least), because the installation runs `sage-location` at the end, which
 hasn't been installed yet. So perhaps `sage -i ...` should only run `sage-
 location` if it already exists.  This way something like
 {{{
 $ make || ./sage -i spkg/standard/zn_poly-0.9.p10.spkg && make ptestlong
 }}}

 might work the way I want, and building Sage would require less
 babysitting.

--

Comment (by jhpalmieri):

 Here's a new patch, using `if [ -x "$SAGE_LOCAL/bin/python" -a -x
 "$SAGE_LOCAL/bin/sage-location" ]...`.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14303#comment:4>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to