#13031: Use cythonize() from cython for Sage module building.
----------------------------------------------------+-----------------------
       Reporter:  robertwb                          |         Owner:  
GeorgSWeber 
           Type:  enhancement                       |        Status:  
needs_review
       Priority:  major                             |     Milestone:  sage-5.4  
  
      Component:  build                             |    Resolution:            
  
       Keywords:  sd40.5                            |   Work issues:  Sphinx    
  
Report Upstream:  N/A                               |     Reviewers:            
  
        Authors:  Robert Bradshaw, R. Andrew Ohana  |     Merged in:            
  
   Dependencies:  #13029                            |      Stopgaps:            
  
----------------------------------------------------+-----------------------
Changes (by rohana):

  * status:  needs_work => needs_review


Comment:

 Ok, rebased on 5.4.beta1.

 The issue with sphinx seems to be because of the python functions defined
 in `sage/ext/gmp.pxi`. We probably shouldn't have any python function
 defined in pxi files anyway (since we currently aren't checking them for
 coverage or anything). There are python functions in other pxi files, but
 they aren't causing any issues for some reason (Robert, maybe you could
 explain why?).

 Anyway, for right now I removed one of them (that wasn't being used
 anywhere), and I moved the other two to `sage/rings/integer.pyx` since
 that is where the sage library looks for them.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13031#comment:23>
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.

Reply via email to