#11321: Make lcalc compatible with the new PARI
--------------------------------------------+-------------------------------
Reporter: jdemeyer | Owner: tbd
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: packages | Keywords: lcalc spkg
Work_issues: | Upstream: Reported
upstream. Developers acknowledge bug.
Reviewer: Volker Braun, Leif Leonhardy | Author: Jeroen Demeyer
Merged: | Dependencies: #11130
--------------------------------------------+-------------------------------
Comment(by leif):
Replying to [comment:27 leif]:
> This still applies:
>
> Replying to [comment:18 leif]: [[BR]]
> > * (GNU) `patch` should be added to the dependencies section. [[BR]]
> > * There's neither a `patches/README.txt` nor a ''Patches'' section in
`SPKG.txt`. [[BR]]
> > * `success 'install'` should perhaps be `success 'copying header
files'`. [[BR]]
>
>
> I'll perhaps add a reviewer patch for this.
Did so. I've also made further cosmetic changes to `spkg-install`, and
also changed the order of flags in `CFLAGS` etc. such that user settings
only get overridden when necessary.
I'll perhaps add a second reviewer patch fixing the hardcoded `g++` in
Lcalc's Makefile, eventually along with a p9 spkg with the other two
superfluous upstream files deleted.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11321#comment:30>
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.
