-
Notifications
You must be signed in to change notification settings - Fork 321
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
rimage/cmake: add find_package(OpenSSL ...) #8349
Conversation
Support unusual locations and deprecates OpenSSL 1 (as previously attempted in thesofproject/rimage#157) Will remove the OpenSSL 1 code later to give people some time to upgrade. Signed-off-by: Marc Herbert <[email protected]>
Without specifying any version, all Windows builds failed like this one https://github.com/thesofproject/sof/actions/runs/6567376740/job/17839955554?pr=8349
"C:/Program Files/OpenSSL/lib/libcrypto.lib" is version "1.1.1w" as found below. It most likely comes from Github: https://github.com/actions/runner-images/blob/win22/20230918.1/images/win/Windows2022-Readme.md Tests on device are OK https://sof-ci.01.org/sofpr/PR8349/build165/devicetest/index.html |
Without specifying any version, https://quickbuild.igk.intel.com/build/12982331 is the same |
1e39f62
to
4e60c9f
Compare
When requesting version 3-3.999
Github's Windows misses the MSYS version and fails like this: https://github.com/thesofproject/sof/actions/runs/6568042556/job/17841835347?pr=8349
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marc-hb I think there is OpenSSL1 C code that also need to be ripped out, IIUC its behind an #ifdef today and is not enabled as default config is OpenSSL3
Yes, this PR would be the first step (blocked by many build systems too old). thesofproject/rimage#157 tried to solve too many problems at once.
Not exactly:
There is no default config in the source, the source code automatically adjusts to whatever OpenSSL version CMake points it at. Both version 1 and version 3 are equally supported with no preference or precedence right now. |
Filed new issue: |
Support unusual locations and prepares for OpenSSL 1 deprecation (as previously attempted in thesofproject/rimage#157)