#11761: Upgrade Cython to 0.15.1
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: tbd
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.7.3
Component: packages | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Jeroen Demeyer | Author: Robert Bradshaw
Merged: | Dependencies:
------------------------------+---------------------------------------------
Changes (by robertwb):
* status: needs_work => needs_review
Comment:
I've fixed the executable bit on spkg-install and added the tests to the
spkg. (I remember someone complaining way back that they were too big, but
I don't think it's an issue and we've regularly shipped them before, and
it's easier to just have them there.) The spkg at the above link has been
refreshed.
The warnings are primarily due to the fact that {{{mpz_t}}} et al are
declared as {{{void*}}} rather than arrays of size 1.
http://hg.sagemath.org/sage-
main/file/2a2abbcad325/sage/libs/gmp/types.pxd#l1 Once this is fixed the
warnings should go away (and they are correct with respect to the current
declarations). All the other warnings I've noticed are correct too, I
simply haven't had the time to fix/update Sage's code.
Your review patch looks fine. You obviously looked at this, is the window
closed for 4.7.2?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11761#comment:20>
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.
