#13963: Fix checking for and resetting GAP workspaces
----------------------------------+-----------------------------------------
       Reporter:  jdemeyer        |         Owner:  GeorgSWeber 
           Type:  defect          |        Status:  needs_work  
       Priority:  critical        |     Milestone:  sage-5.7    
      Component:  build           |    Resolution:              
       Keywords:                  |   Work issues:              
Report Upstream:  N/A             |     Reviewers:  Volker Braun
        Authors:  Jeroen Demeyer  |     Merged in:              
   Dependencies:                  |      Stopgaps:              
----------------------------------+-----------------------------------------

Comment (by leif):

 I'd still let `-O0` override a user's O-level (in `$CFLAGS`) if
 `SAGE_DEBUG=yes`, like most -- if not all -- other spkgs do.

 And while you're at it, change the `==` to `=` there, and/or use `[[ ...
 ]]`.

 There are also a few error checks missing, and `SPKG_ROOT` isn't always
 quoted.  (One should also probably do some sanity check on `$VERSION` at
 least.)

 The `-f` in `ln -sf ... latest` is redundant, as we always first delete
 the symbolic link (if present).

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13963#comment:21>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to