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

Don't automatically detect libssl/libcrypto with ExtUtils::PkgConfig #466

Conversation

chrisnovakovic
Copy link
Collaborator

This reverts da6292e.

The automatic detection of libssl/libcrypto with pkg-config conflicts with the OPENSSL_PREFIX method of specifying where Makefile.PL should look for libssl/libcrypto. The one known failure case at the moment is when OPENSSL_PREFIX points to a copy of OpenSSL that has compression enabled, but pkg-config points to a copy of OpenSSL that doesn't have compression enabled - this causes -lz to be omitted from the list of options passed to the linker, causing a compilation failure. There may be other failure cases.

The automatic detection of libssl/libcrypto is a very useful feature, so we'll find and fix all the edge cases and bring it back in time for Net-SSLeay 1.96.

Fixes #465.

This reverts da6292e.

The automatic detection of libssl/libcrypto with pkg-config conflicts
with the OPENSSL_PREFIX method of specifying where Makefile.PL should
look for libssl/libcrypto. The one known failure case at the moment is
when OPENSSL_PREFIX points to a copy of OpenSSL that has compression
enabled, but pkg-config points to a copy of OpenSSL that doesn't have
compression enabled - this causes "-lz" to be omitted from the list of
options passed to the linker, causing a compilation failure. There may
be other failure cases.

The automatic detection of libssl/libcrypto is a very useful feature, so
we'll find and fix all the edge cases and bring it back in time for
Net-SSLeay 1.96.

Fixes radiator-software#465.
@chrisnovakovic chrisnovakovic added the bug Something isn't working label Jan 6, 2024
@chrisnovakovic chrisnovakovic self-assigned this Jan 6, 2024
@chrisnovakovic chrisnovakovic merged commit 2345a30 into radiator-software:master Jan 6, 2024
441 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong linker options used when ExtUtils::PkgConfig is installed
1 participant