Skip to content

Close #10 #7: add pkg-config install for brew#13

Open
ReedL wants to merge 1 commit into
luckybulldozer:masterfrom
ReedL:ReedL-installer-patch-1
Open

Close #10 #7: add pkg-config install for brew#13
ReedL wants to merge 1 commit into
luckybulldozer:masterfrom
ReedL:ReedL-installer-patch-1

Conversation

@ReedL

@ReedL ReedL commented Apr 7, 2015

Copy link
Copy Markdown

The line in the install script output reported by two of the users (and experienced by me):
"make: /usr/local/bin/pkg-config: Command not found"
Caused me to make a link from my pkg-config I already had to /usr/local/bin/pkg-config. This didn't help much as that copy of pkg-config had no idea where all the .pc library files were. As I started manually adding the paths to the .pc files needed to pkg-config's search path, I eventually realized that there had to be a much better way. On a hunch, I installed a new copy of pkg-config from brew and successfully compiled visualSFM.

Because the pkg-config installed by brew needs to be run, I suspect that #12 might just be using the wrong pkg-config and possibly fixed by this change as well.

Edit: I have not used github before, apologies in advance.

Close #10 #7

…brew

The line in the install script output reported by two of the users (and experienced by me): 
"make: /usr/local/bin/pkg-config: Command not found"
Caused me to make a link from my pkg-config I already had to /usr/local/bin/pkg-config. This didn't help much as that copy of pkg-config had no idea where all the .pc library files were. As I started manually adding the paths to the .pc files needed to pkg-config's search path, I eventually realized that there had to be a much better way. On a hunch, I installed a new copy of pkg-config from brew and successfully compiled visualSFM.

Because the pkg-config installed by brew needs to be run, I suspect that luckybulldozer#12 might just be using the wrong pkg-config and possibly fixed by this change as well.

Close luckybulldozer#10 luckybulldozer#7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make error on Yousemite

1 participant