./configure: On OSX, if brew present, use /usr/local instead of /opt/local #35

Merged
merged 1 commit into from May 15, 2013

Conversation

Projects
None yet
4 participants
@kennknowles
Contributor

kennknowles commented Aug 23, 2012

A quick check for the most obvious Homebrew scenario, and corresponding path adjustment.

@anthonybishopric

This comment has been minimized.

Show comment Hide comment
@anthonybishopric

anthonybishopric Apr 1, 2013

+1 for this.

~/src/pfff → master]$ ./configure --prefix=/usr/local
Checking your configuration.

OCaml (the wonderful language) is present.

make (gnu version) is present.
grep: /opt/local/include/pcre.h: No such file or directory
!!!! The PCRE library is missing. !!!!

But I have installed pcre via homebrew.

+1 for this.

~/src/pfff → master]$ ./configure --prefix=/usr/local
Checking your configuration.

OCaml (the wonderful language) is present.

make (gnu version) is present.
grep: /opt/local/include/pcre.h: No such file or directory
!!!! The PCRE library is missing. !!!!

But I have installed pcre via homebrew.

@john-difool

This comment has been minimized.

Show comment Hide comment
@john-difool

john-difool May 2, 2013

You need to update your configure and change the location of PCRE on Mac/HB:

if($arch =~ /Darwin/) {
    $PCRE_INCLUDE="/usr/local/include";
    $PCRE_LIBS="/usr/local/lib";
}

then configure will be happy (2nd line should tell you that PCRE is detected).

You need to update your configure and change the location of PCRE on Mac/HB:

if($arch =~ /Darwin/) {
    $PCRE_INCLUDE="/usr/local/include";
    $PCRE_LIBS="/usr/local/lib";
}

then configure will be happy (2nd line should tell you that PCRE is detected).

@kennknowles

This comment has been minimized.

Show comment Hide comment
@kennknowles

kennknowles May 2, 2013

Contributor

@ldesegur that is exactly what this patch does. It also adjusts for homebrew vs macports.

Contributor

kennknowles commented May 2, 2013

@ldesegur that is exactly what this patch does. It also adjusts for homebrew vs macports.

aryx added a commit that referenced this pull request May 15, 2013

Merge pull request #35 from kennknowles/homebrew-osx-paths
./configure: On OSX, if brew present, use /usr/local instead of /opt/local

@aryx aryx merged commit ba7e861 into facebook:master May 15, 2013

@kennknowles kennknowles deleted the kennknowles:homebrew-osx-paths branch May 15, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment