#12680: Fix (a single) hardcoded 'g++' in Singular 3-1-3-3
------------------------+---------------------------------------------------
   Reporter:  leif      |          Owner:  leif                                 
      
       Type:  defect    |         Status:  needs_review                         
      
   Priority:  major     |      Milestone:  sage-5.0                             
      
  Component:  packages  |       Keywords:  spkg CXX C++ compiler hard-coded     
      
Work_issues:            |       Upstream:  Not yet reported upstream; Will do 
shortly.
   Reviewer:            |         Author:  Leif Leonhardy                       
      
     Merged:            |   Dependencies:                                       
      
------------------------+---------------------------------------------------
Changes (by newvalueoldvalue):

  * status:  new => needs_review
  * author:  => Leif Leonhardy


Old description:

> In `src/factory/GNUmakefile.in`, we currently have:
> {{{
> #!make
> # create cf_gmp.h
> cf_gmp.h: GNUmakefile gen_cf_gmp.template
>         echo "#!/bin/sh" >gen_cf_gmp.sh
>         echo "GMP_H_T=\"`g++ $(LIBCXXFLAGS) -M gen_cf_gmp.cc|grep gmp.h`
> \"" >>gen_cf_gmp.sh
>         cat gen_cf_gmp.template >>gen_cf_gmp.sh
>         $(SHELL) gen_cf_gmp.sh
> }}}
>
> The attached p6 spkg fixes this (by using `$(CXX)`, as everywhere else),
> and also two typos.
>
> ----
>
> This bug shows up (and leads to a build error) e.g. if you use a
> different C++ compiler (or version of `g++`) by setting `CXX` and specify
> `CXXFLAGS` the "default" `g++` doesn't understand.

New description:

 In `src/factory/GNUmakefile.in`, we currently have:
 {{{
 #!make
 # create cf_gmp.h
 cf_gmp.h: GNUmakefile gen_cf_gmp.template
         echo "#!/bin/sh" >gen_cf_gmp.sh
         echo "GMP_H_T=\"`g++ $(LIBCXXFLAGS) -M gen_cf_gmp.cc|grep gmp.h`
 \"" >>gen_cf_gmp.sh
         cat gen_cf_gmp.template >>gen_cf_gmp.sh
         $(SHELL) gen_cf_gmp.sh
 }}}

 The attached p6 spkg fixes this (by using `$(CXX)`, as everywhere else),
 and also two typos.

 ----

 This bug shows up (and leads to a build error) e.g. if you use a different
 C++ compiler (or version of `g++`) by setting `CXX` and specify `CXXFLAGS`
 the "default" `g++` doesn't understand.

 ----

 '''New spkg:'''
 
[http://boxen.math.washington.edu/home/leif/Sage/spkgs/singular-3-1-3-3.p6.spkg]

 '''md5sum:''' `c637e93eff14491d3dff11647b4732af  singular-3-1-3-3.p6.spkg`

 === singular-3-1-3-3.p6 (Leif Leonhardy, March 17th 2012) ===
  * #12680: Fix hardcoded 'g++' (and two typos) in factory/GNUmakefile.in.

--

Comment:

 Haven't yet looked whether there are other changes pending; I apologize
 for that.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12680#comment:1>
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