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

CORE-4264: cmake: Change LD_LIBRARY_PATH for OpenSSL #19942

Conversation

michael-redpanda
Copy link
Contributor

Now that OpenSSL libraries are installed in lib, update OPENSSL_ENV to use correct library path.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.1.x
  • v23.3.x
  • v23.2.x

Release Notes

  • None

@michael-redpanda michael-redpanda requested a review from a team June 21, 2024 12:12
@michael-redpanda michael-redpanda self-assigned this Jun 21, 2024
@michael-redpanda michael-redpanda requested review from pgellert and removed request for a team June 21, 2024 12:12
@michael-redpanda michael-redpanda requested review from BenPope and a team as code owners June 21, 2024 12:12
@michael-redpanda michael-redpanda requested review from andrewhsu and removed request for a team June 21, 2024 12:12
@michael-redpanda
Copy link
Contributor Author

FYI build here is going to fail as it depends on https://github.com/redpanda-data/vtools/pull/2875

BenPope
BenPope previously approved these changes Jun 21, 2024
Copy link
Member

@BenPope BenPope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

pgellert
pgellert previously approved these changes Jun 21, 2024
Now that OpenSSL libraries are installed in lib, update OPENSSL_ENV to
use correct library path.

Also update all tests to look in `lib` rather than `lib64` for modules.

Signed-off-by: Michael Boquard <michael@redpanda.com>
@michael-redpanda
Copy link
Contributor Author

/ci-repeat 1

@michael-redpanda
Copy link
Contributor Author

@michael-redpanda michael-redpanda merged commit 56bf7f4 into redpanda-data:dev Jun 22, 2024
14 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants