#12682: Fix "hardcoded" 'gcc' in ratpoints 2.1.3 [p2]
------------------------+---------------------------------------------------
   Reporter:  leif      |          Owner:  leif                       
       Type:  defect    |         Status:  new                        
   Priority:  major     |      Milestone:  sage-5.0                   
  Component:  packages  |       Keywords:  spkg CC compiler hard-coded
Work_issues:            |       Upstream:  N/A                        
   Reviewer:            |         Author:                             
     Merged:            |   Dependencies:                             
------------------------+---------------------------------------------------
 The upstream `Makefile` defines `CC` to `gcc`, which could be overridden
 by using `make CC="$CC" ...` in `spkg-install` (similar for `INSTALL_DIR`,
 which the `Makefile` defines to `/usr/local`).

 The updated spkg patches the `Makefile` to let `CC` just ''default'' to
 `gcc`, and take the value of the environment variable `CC` if the latter
 is defined (which is ''always'' the case in Sage, but that doesn't
 matter).  Similar for `INSTALL_DIR`, such that `spkg-install` can (and
 does now) make use of the `install-lib` target, also quoting the target
 filenames to allow spaces in `$SAGE_LOCAL`.

 I've also cleaned up `SPKG.txt` and `spkg-install` a little; the latter
 now also uses `$MAKE` instead of `make`.

 I'm not sure whether we should add running some tests (there's a `test`
 target which does use the built library), or even add an `spkg-check`
 script; if so, perhaps on a follow-up ticket.  [See also my comment in
 `SPKG.txt`.]

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12682>
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