Opened 11 years ago
Closed 11 years ago
#12680 closed defect (fixed)
Fix (a single) hardcoded 'g++' in Singular 3-1-3-3
Reported by: | leif | Owned by: | leif |
---|---|---|---|
Priority: | major | Milestone: | sage-5.0 |
Component: | packages: standard | Keywords: | rd2 spkg CXX C++ compiler hard-coded |
Cc: | AlexanderDreyer, malb, ohanar | Merged in: | sage-5.0.beta9 |
Authors: | Leif Leonhardy | Reviewers: | R. Andrew Ohana |
Report Upstream: | Completely fixed; Fix reported upstream | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
In src/factory/GNUmakefile.in
, we currently have:
# 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.
Attachments (2)
Change History (9)
Changed 11 years ago by
Attachment: | singular-3-1-3-3.p5-p6.diff added |
---|
comment:1 Changed 11 years ago by
Authors: | → Leif Leonhardy |
---|---|
Description: | modified (diff) |
Status: | new → needs_review |
Haven't yet looked whether there are other changes pending; I apologize for that.
Changed 11 years ago by
Attachment: | factory.GNUmakefile.in.patch added |
---|
This is the new patch to upstream. For reference / review only.
comment:2 follow-up: 3 Changed 11 years ago by
Reviewers: | → R. Andrew Ohana |
---|---|
Status: | needs_review → positive_review |
looks good and works well!
comment:3 Changed 11 years ago by
Replying to ohanar:
looks good and works well!
Thanks for the review!
Perhaps Alexander (or Martin) wants to report this upstream, as their trac occasionally doesn't like me... ;-) (Haven't yet tried to, though.)
comment:4 Changed 11 years ago by
Keywords: | rd2 added |
---|
comment:5 Changed 11 years ago by
Report Upstream: | Not yet reported upstream; Will do shortly. → Completely fixed; Fix reported upstream |
---|
Replying to leif:
Perhaps Alexander (or Martin) wants to report this upstream, as their trac occasionally doesn't like me... ;-) (Haven't yet tried to, though.)
Ok, this is now Singular trac ticket #413.
What's "rd2"? Rohana Days II?
comment:7 Changed 11 years ago by
Merged in: | → sage-5.0.beta9 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Diff between the p5 and my p6. For reference / review only.