Skip to content

[OSX] fix brew openssl detection#7560

Closed
jonasschnelli wants to merge 1 commit intobitcoin:masterfrom
jonasschnelli:2016/02/osx_openssl
Closed

[OSX] fix brew openssl detection#7560
jonasschnelli wants to merge 1 commit intobitcoin:masterfrom
jonasschnelli:2016/02/osx_openssl

Conversation

@jonasschnelli
Copy link
Copy Markdown
Contributor

ping @theuni (not sure if I did this correct, it seems like extending PKG_CONFIG_PATH does not work for a later header check with AC_CHECK_HEADER).

OSX slowly depractates openssl. Since 10.11.3, not event Xcode installs the openssl headers. This PR will allow using homebrews openssl headers.

@jameshilliard
Copy link
Copy Markdown
Contributor

The workaround I've used is:

brew link openssl --force

@jonasschnelli
Copy link
Copy Markdown
Contributor Author

The force linking is also a good idea, though, I think if we detect brews OpenSSL pkgconf, we should also allow to detect the headers in brews OpenSSL path.

@jameshilliard
Copy link
Copy Markdown
Contributor

I think this is how you would do that properly.

@jonasschnelli
Copy link
Copy Markdown
Contributor Author

@jameshilliard I think this PR would do it exactly as recommended in you link.

@jameshilliard
Copy link
Copy Markdown
Contributor

Doesn't it need $(brew --prefix) in the include path for keg only openssl?

@jonasschnelli
Copy link
Copy Markdown
Contributor Author

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Feb 29, 2016

@theuni can you take a look here?

@theuni
Copy link
Copy Markdown
Member

theuni commented Mar 1, 2016

@jonasschnelli hmm, appending PKG_CONFIG_PATH should be enough to pick up on the openssl headers (that's what it's there for). In the header check, we first set CFLAGS="$CFLAGS $SSL_CFLAGS $CRYPTO_CFLAGS", which should add whatever pkg-config finds.

So your change shouldn't be necessary, seems to me something else is broken.

Could you please paste the relevant failure from config.log?

@jonasschnelli
Copy link
Copy Markdown
Contributor Author

Cannot reproduce the issue anymore. It was probably a local-only thing.
Thanks @theuni!

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants