#13432: add sage/env.py and fix many inappropriate references to SAGE_ROOT
-----------------------------------+----------------------------------------
       Reporter:  rohana           |         Owner:  jason       
           Type:  task             |        Status:  needs_review
       Priority:  major            |     Milestone:  sage-5.9    
      Component:  misc             |    Resolution:              
       Keywords:                   |   Work issues:              
Report Upstream:  N/A              |     Reviewers:              
        Authors:  R. Andrew Ohana  |     Merged in:              
   Dependencies:  #13123, #13348   |      Stopgaps:              
-----------------------------------+----------------------------------------

Comment (by fbissey):

 Thank you for the new patch. I will still whine a bit about setup.py. In
 the definition of "include_dirs" on line 54 (after patch) you didn't put
 SAGE_INC itself. I think the list should be
 {{{
 include_dirs = [SAGE_INC,
                 os.path.join(SAGE_INC, 'csage'),
                       os.path.join(SAGE_SRC, 'sage', 'ext')]
 }}}
 I don't think we can count on CPATH being defined and providing SAGE_INC
 in all cases so I think it needs to be included here.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13432#comment:25>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to