#13031: Use cythonize() from cython for Sage module building.
----------------------------------------------------+-----------------------
Reporter: robertwb | Owner:
GeorgSWeber
Type: enhancement | Status:
needs_work
Priority: major | Milestone: sage-5.1
Component: build | Resolution:
Keywords: sd40.5 | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Robert Bradshaw, R. Andrew Ohana | Merged in:
Dependencies: #13029 | Stopgaps:
----------------------------------------------------+-----------------------
Changes (by {'newvalue': u'Robert Bradshaw, R. Andrew Ohana', 'oldvalue': ''}):
* status: needs_review => needs_work
* author: => Robert Bradshaw, R. Andrew Ohana
Comment:
I attached an updated patch that
* is rebased on the final 5.1.beta2
* fixes a `ZeroDivisionError` that was preventing documentation from
building
* more throughly cleans up `module_list.py` with the use of *s
* does other clean up with `module_list.py` since this patch will
constantly need to be rebased with each development release (may as well
clean up everything while we are at it)
That said, there is still some issue with Cython's cythonize, even using
the old `module_list.py` you encounter the issue:
{{{
sage: cython('a = 5')
sage: a
Traceback (most recent call last):
...
NameError: name 'a' is not defined
}}}
You'll find that tons of doctests fail, and frequently they are having
`NameError`s like this (although they aren't necessarily calling Sage's
cython function).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13031#comment:8>
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.
