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

Brew fix for openssl in default path #11436

Closed
wants to merge 2 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Jul 13, 2023

  • refs builds with libressl on macOS fail #11413
  • if brew install/update links openssl into /usr/local, it will be found before anything we add with -isystem path to CPP/LDLFAGS. Get rid of that by unlinking the keg.

- refs curl#11413
- if brew install/update links openssl into /usr/local, it will be
  found before anything we add with `-isystem path` to CPP/LDLFAGS.
  Get rid of that by unlinking the keg.
@github-actions github-actions bot added the CI Continuous Integration label Jul 13, 2023
@bagder
Copy link
Member

bagder commented Jul 13, 2023

✔️ !

@bagder
Copy link
Member

bagder commented Jul 13, 2023

Thanks!

@bagder bagder closed this in 5c07439 Jul 13, 2023
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
If brew install/update links openssl into /usr/local, it will be found
before anything we add with `-isystem path` to CPP/LDLFAGS.  Get rid of
that by unlinking the keg.

Fixes curl#11413
Closes curl#11436
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
If brew install/update links openssl into /usr/local, it will be found
before anything we add with `-isystem path` to CPP/LDLFAGS.  Get rid of
that by unlinking the keg.

Fixes curl#11413
Closes curl#11436
@stefankiesz
Copy link

Hi, does anyone know what's the cause of this behavior? When I build curl using cmake, it finds the correct openssl version with find_package, but still fails because the linker tries to link with the brew installed openssl.

Is there a way to fix this without unlinking?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Development

Successfully merging this pull request may close these issues.

None yet

3 participants