#9208: Add PKG_CONFIG_PATH to sage-env so programs like matplotlib link 
properly.
----------------------------+-----------------------------------------------
   Reporter:  drkirkby      |       Owner:  drkirkby    
       Type:  defect        |      Status:  needs_review
   Priority:  major         |   Milestone:  sage-4.4.4  
  Component:  build         |    Keywords:              
     Author:  David Kirkby  |    Upstream:  N/A         
   Reviewer:                |      Merged:              
Work_issues:                |  
----------------------------+-----------------------------------------------

Comment(by jason):

 Replying to [comment:7 drkirkby]:
 > I meant to overwrite the patch when I corrected the spelling, but
 forgot, so there is now 9208.2.patch.
 >
 > Are you suggesting this gets reviewed and incorporated into Sage and you
 write another patch which rewrites sage-env if Sage is moved?

 Not sage-env.  See #9210 for the patch I talked about.

 > If so, then this still needs reviewing.
 >

 Yes.  Since I practically had the same patch, but you beat me to posting
 it, I think the review will be easy.

 I didn't check for the existence of PKG_CONFIG_PATH, though.  In what
 situation will we not want to overwrite that variable?

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