#11680: support extra_compile_args (e.g., C99) when loading/attaching .pyx 
(cython)
files, and when using %cython in the notebook
---------------------------+------------------------------------------------
   Reporter:  was          |          Owner:  jason          
       Type:  enhancement  |         Status:  needs_review   
   Priority:  minor        |      Milestone:  sage-4.7.2     
  Component:  misc         |       Keywords:                 
Work_issues:               |       Upstream:  N/A            
   Reviewer:               |         Author:  Martin Albrecht
     Merged:               |   Dependencies:                 
---------------------------+------------------------------------------------

Comment(by leif):

 Hmmm, the naming isn't very clear.

 I'd use some plural form for `cfile`, i.e. `cfiles`, or something else,
 e.g. `csources`, which clearly indicates that these are additional,
 "arbitrary" source files to also be compiled into the extension module.

 `cargs` is similarly ambiguous, as the `c` refers to Cython, not C, but
 the "args" are passed to the C or C++ compiler rather than to `cython`
 itself.

 (I admit that `clib` and `cinclude` could perhaps have better names, or at
 least the plural form, as well; especially for the latter it isn't clear
 that its parameters are or can be actually ''search paths''.)

 [[BR]]

 For compiling C99, I still prefer having `#clang C99` (case-insensitive
 for all languages btw.).

 P.S.: Now I'm happy ''I'' didn't also work on this yesterday... ;-)

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

Reply via email to