Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

restore x11 to pkg-config's list of default paths #13457

Closed
wants to merge 1 commit into from
Closed

restore x11 to pkg-config's list of default paths #13457

wants to merge 1 commit into from

Conversation

nolta
Copy link

@nolta nolta commented Jul 17, 2012

You used to be able to install matplotlib with

$ brew install pkg-config
$ easy_install matplotlib

But 63415cc broke this, by removing /usr/X11/lib/pkgconfig from pkg-config's list of default paths. Setting PKG_CONFIG_PATH doesn't help, due to a bug in matplotlib's makefiles.

@jacknagel
Copy link
Contributor

This was removed very deliberately, and is not the way to address this issue.

@nolta
Copy link
Author

nolta commented Jul 18, 2012

I know, i read the #12607 thread.

But given that brew doctor used to report

Error: Your pkg-config is not checking "/usr/X11/lib/pkgconfig" for packages.

it's not surprising that people came to rely on this behavior.

Would you consider a patch to brew doctor, stating that the behavior of pkg-config has changed, and that they might need to add

export PKG_CONFIG_PATH=/opt/X11/lib/pkgconfig:/usr/X11/lib/pkgconfig

to their .bashrc?

@jacknagel
Copy link
Contributor

No, persistent warnings are bad. Users who don't care about this and don't set that variable will always see the warning.

@Sharpie
Copy link
Contributor

Sharpie commented Jul 18, 2012

Setting PKG_CONFIG_PATH doesn't help, due to a bug in matplotlib's makefiles.

Any progress on this front? Seems like the most direct resolution is for matplotlib to fix their bug.

@adamv
Copy link
Contributor

adamv commented Jul 23, 2012

Should be fixed upstream.

MacPorts packages matplotlib, but patches it to use proper folders: https://trac.macports.org/browser/trunk/dports/python/py-matplotlib/files/patch-setupext.py.diff

@adamv adamv closed this Jul 23, 2012
@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants