#12681: Fix hardcoded 'g++' in Lcalc 1.23 [p9]
------------------------+---------------------------------------------------
Reporter: leif | Owner: leif
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: packages | Keywords: spkg CC CXX C++ compiler hard-coded
Work_issues: | Upstream: N/A
Reviewer: | Author: Leif Leonhardy
Merged: | Dependencies:
------------------------+---------------------------------------------------
Comment(by leif):
Replying to [comment:2 mjo]:
> Same comment as on ratpoints: I think we should avoid adding patches
unless absolutely necessary.
Lcalc is a special case in many ways (cf. my reply on #12682), and we have
to patch the Makefile anyway (since there's still no `configure`, and
therefore the Makefile is supposed to get edited).
Michael Rubinstein would certainly be happy if we sent him some generic
solution, but so far nobody found the time for such, as mentioned.
[[BR]]
> (Using `$CXX` instead of `g++` is obviously correct; although we should
report that upstream too.)
We already did so, IIRC...
Btw., haven't yet looked at the 1.3 (beta) version, which I think is still
work in progress.
We can certainly contribute to that, although e.g. Jeroen rejected to make
patches more upstream-friendly, i.e. generic (w.r.t. supporting different
PARI versions). Also took me some time to convince people to use `-lgmp`
instead of `-lmpir`.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12681#comment:3>
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.
