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

crypto/cmake: Fixed exact versioning #16936

Conversation

michael-redpanda
Copy link
Contributor

@michael-redpanda michael-redpanda commented Mar 7, 2024

Will not look for an exact version in the OSS build

Fixes: #16932

Look to https://github.com/redpanda-data/vtools/pull/2552 to see how the exact OpenSSL version is selected

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
  • v23.3.x
  • v23.2.x

Release Notes

  • none

Copy link
Contributor

@rockwotj rockwotj left a comment

Choose a reason for hiding this comment

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

This LGTM assuming it works. I left some comments on the vtools side of things

@@ -1,4 +1,9 @@
find_package(OpenSSL 3.0.9 EXACT REQUIRED)
if (RP_OPENSSL_VERSION)
message(STATUS "Searching for exactly OpenSSL version ${RP_OPENSSL_VERSION}")
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we really need this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i can remove it, was just a nice way of verifying it was working :)

Will not look for an exact version in the OSS build

Fixes: redpanda-data#16932

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

Force push 72f9a9a:

  • Removed debug print statement

@michael-redpanda michael-redpanda merged commit 33601c4 into redpanda-data:dev Mar 7, 2024
15 checks passed
@dotnwat
Copy link
Member

dotnwat commented Mar 8, 2024

thanks!

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.

Open source build breaks on OpenSSL
3 participants