#10260: module_list.py: quaternion_algebra_element.pyx wrongly uses -std=c99 
option
------------------------------+---------------------------------------------
   Reporter:  jdemeyer        |       Owner:  GeorgSWeber    
       Type:  defect          |      Status:  positive_review
   Priority:  major           |   Milestone:  sage-4.6.1     
  Component:  build           |    Keywords:                 
     Author:  Jeroen Demeyer  |    Upstream:  N/A            
   Reviewer:                  |      Merged:                 
Work_issues:                  |  
------------------------------+---------------------------------------------

Comment(by jdemeyer):

 Replying to [comment:2 fbissey]:
 > Tell you what. I introduced exactly the same patch in sage-on-gentoo at
 the end of June. While you are at it can we fix this one as well:
 > {{{
 >     Extension('sage.rings.polynomial.polynomial_rational_flint',
 >               sources =
 ['sage/rings/polynomial/polynomial_rational_flint.pyx',
 'sage/libs/flint/fmpq_poly.c'],
 >               language = 'c++',
 >               extra_compile_args=["-std=c99"] + uname_specific('SunOS',
 [], ['-D_XPG6']),
 >               libraries = ["csage", "flint", "ntl", "gmpxx", "gmp"],
 >               include_dirs = [SAGE_ROOT + '/local/include/FLINT/',
 SAGE_ROOT + '/devel/sage/sage/libs/flint/'],
 >               depends = [SAGE_ROOT + "/local/include/FLINT/flint.h"]),
 > }}}

 This one is harder, because there are ''two'' files, one C99 and one C++.
 So we cannot simply remove {{{-std=c99}}} there.

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