#13031: Use cythonize() from cython for Sage module building.
----------------------------------------------------+-----------------------
Reporter: robertwb | Owner:
GeorgSWeber
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.3
Component: build | Resolution:
Keywords: sd40.5 | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Robert Bradshaw, R. Andrew Ohana | Merged in:
Dependencies: #13029 | Stopgaps:
----------------------------------------------------+-----------------------
Old description:
New description:
Cython's cythonize is now robust enough to replace our custom dependency
logic, with a few extra features such as * syntax for cython extensions
and cycache (still needs work).
Apply:
* [attachment:trac13031.patch]
--
Comment (by rohana):
I just rebased on 5.2.beta1 since the dependency was merged into that
release. However, in the process I discovered that cycache is not ready
for primetime yet:
{{{
$ grep 'cdef class MonoDict:' sage -R
sage/sets/disjoint_set.c: * cdef class MonoDict: #
<<<<<<<<<<<<<<
sage/matrix/matrix_modn_dense_float.cpp: * cdef class MonoDict:
# <<<<<<<<<<<<<<
}}}
Neither of these modules changed between 5.1.beta2 and 5.2.beta1, but
their dependencies changed, so there is some issue with the hashing that
needs to be resolved.
For now I've disabled cycache, since it will be trivial to re-add once
this ticket gets merged.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13031#comment:14>
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.
