#11321: Make lcalc compatible with the new PARI
--------------------------------------------+-------------------------------
Reporter: jdemeyer | Owner: tbd
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.1
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):
From first glance:
* (GNU) `patch` should be added to the dependencies section.
* This is still in:
{{{
#!sh
# If SAGE_DEBUG is set either unset (the default), or set to 'yes'
...
echo >&2 "Code will be built with debugging information present. Set
'SAGE_DEBUG' to 'no' if you don't want that."
# Actually anything othe than 'yes' will cause
# no debugging information to be added.
}}}
(As mentioned, I also would add `-g` by default.)
* There's neither a `patches/README.txt` nor a ''Patches'' section in
`SPKG.txt`.
* `success 'install'` should perhaps be `success 'copying header files'`.
The rest looks ok.
I've btw. successfully tested the ''previous p7'' with PARI 2.5.0.p0 and
Sage 4.7.1.rc0 (Ubuntu 10.04 x86_64, GCC 4.5.1), both with our current
MPIR and the newer MPIR 2.1.3 from #8664.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11321#comment:18>
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.
