#11765: Sage's 'pkg-config' directory should always be added to PKG_CONFIG_PATH
------------------------------+---------------------------------------------
   Reporter:  leif            |          Owner:  leif                  
       Type:  defect          |         Status:  positive_review       
   Priority:  blocker         |      Milestone:  sage-4.7.2            
  Component:  scripts         |       Keywords:  pkgconfig .pc sage-env
Work_issues:                  |       Upstream:  N/A                   
   Reviewer:  Steven Trogdon  |         Author:  Leif Leonhardy        
     Merged:                  |   Dependencies:                        
------------------------------+---------------------------------------------

Old description:

> Currently, `PKG_CONFIG_PATH` is only '''set''' to Sage's `pkg-config`
> directory (`$SAGE_ROOT/local/lib/pkgconfig/`) if it is empty (or not set
> at all).
>
> The patch now also prepends Sage's directory otherwise, such that Sage's
> `.pc` files will be picked up first.
>
> The previous behaviour could obviously break the build if
> `PKG_CONFIG_PATH` was already set in the "global" environment, i.e.
> outside the Sage environment.
>
> ----
>
> Apply [attachment:trac_11765-always_add_sages_pc_dir_in_sage-
> env.scripts.patch] to the '''scripts repo'''.

New description:

 Currently, `PKG_CONFIG_PATH` is only '''set''' to Sage's `pkg-config`
 directory (`$SAGE_ROOT/local/lib/pkgconfig/`) if it is empty (or not set
 at all).

 The patch now also prepends Sage's directory otherwise, such that Sage's
 `.pc` files will be picked up first.

 The previous behaviour could obviously break the build if
 `PKG_CONFIG_PATH` was already set in the "global" environment, i.e.
 outside the Sage environment.

 ----

 Apply [attachment:trac_11765-always_add_sages_pc_dir_in_sage-
 env.rebased.scripts.patch] to the '''scripts repo'''.

--

Comment(by leif):

 Replying to [comment:7 fbissey]:
 > Well thanks that saves me the work :)

 If it makes you happy... ;-)

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