Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pkg-config doesn't pickup the homebrew pkg-config path #539

Closed
andyli opened this issue Jul 17, 2016 · 5 comments
Closed

pkg-config doesn't pickup the homebrew pkg-config path #539

andyli opened this issue Jul 17, 2016 · 5 comments

Comments

@andyli
Copy link

andyli commented Jul 17, 2016

Discovered from our TravisCI build that suddenly fails recently.

In the build, after migrating Homebrew to v0.9.9, the pkg-config path search path is not updated:

$ pkg-config --variable pc_path pkg-config
/usr/local/lib/pkgconfig:/usr/local/share/pkgconfig:/usr/lib/pkgconfig:/usr/local/Library/ENV/pkgconfig/10.9

where /usr/local/Library/ENV/pkgconfig/10.9 should be /usr/local/Library/Homebrew/os/mac/pkgconfig/10.9.

@ilovezfs
Copy link
Contributor

You need to brew update. This was fixed.
3560185

@andyli
Copy link
Author

andyli commented Jul 17, 2016

But my build log shows that brew was upgraded to 092d471, which was the most recent commit as of writing.

==> Migrating Homebrew to v0.9.9
remote: Counting objects: 350, done.
remote: Compressing objects: 100% (276/276), done.
remote: Total 350 (delta 169), reused 170 (delta 56), pack-reused 0
Receiving objects: 100% (350/350), 496.62 KiB | 0 bytes/s, done.
Resolving deltas: 100% (169/169), completed with 164 local objects.
From https://github.com/Homebrew/brew
 + 73a2e75...092d471 master     -> origin/master  (forced update)

@ilovezfs
Copy link
Contributor

@andyli It looks like your problem may be here:
https://github.com/andyli/neko/blob/master/.travis.yml#L44-L49

It doesn't upgrade anything but cmake, and the path is hard coded in the pkg-config binary, so you need to at least upgrade pkg-config too, but I'd suggest just making it brew upgrade and see what else is outdated in that image.

andyli added a commit to andyli/neko that referenced this issue Jul 17, 2016
@andyli
Copy link
Author

andyli commented Jul 17, 2016

Doing brew upgrade pkg-config after brew update fixed it. Thanks!
FYI, I tried brew upgrade too, which worked, but it took too long in TravisCI.

@ilovezfs
Copy link
Contributor

@andyli that's great! Glad you got it working.

@Homebrew Homebrew locked and limited conversation to collaborators May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants