-
Notifications
You must be signed in to change notification settings - Fork 66
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
[hotfix] cmake_dependent_option requires CMake 3.25 #251
Conversation
Signed-off-by: Uilian Ries <[email protected]>
Checking the 0.11 in Github action: The option for TLS is ON: But, the configuration results as OFF: And it's Ubuntu. |
Might make sense to revert out the change that is causing this and rework that PR since it looks like cmaked 3.25 isn't available on a lot of the CI runs out of the box. And then probably a v0.11.1 release without it? Side question, would it make sense to have a GitHub action that triggers on release to build for Conan? |
I believe this should resolve the bug, I will subsequently be opening another PR to try an reinvestigate this. |
Would you mind giving that branch a try with the conan build process to verify its working properly? |
Signed-off-by: Uilian Ries <[email protected]>
Yes, give me some minutes. I'll test here and post the result.
Yes, please. In Conan we had some resistance to move to 3.15. Now imagine 3.23 ...
Not needed, usually users push a new package as soon they require :) |
Thanks for reporting the bug, and sorry for the problem! The change seemed harmless but clearly it needs some more testing around it. I'm about to head out skiing for the day so I'll check back in later to make sure everything on v0.11.1 is working well. |
Working with Conan:
|
@jbaldwin No problem at all! Thank you for your quick hotfix! Have a great exercise ⛷️ ! |
Hello!
We are packaging the version 0.11 in Conan and we found an error related to
cmake_dependent_option
when building on Linux. It's possible to reproduce it locally, but basically, does not matter what option value you pass when running CMake <3.25, it will always use the forced negative value. For instance:As you can see,
LIBCORO_FEATURE_TLS
is configured to beON
, but results inOFF
. The code is clear about the condition:cmake_dependent_option(LIBCORO_FEATURE_TLS "Include TLS encryption features, Default=ON." ON "NOT EMSCRIPTEN; LINUX" OFF)
If Not Emscripten AND Is Linux ... That's my current scenario. Still, does not work.
Now using CMake 3.25:
Now it works when running CMake 2.25. I changed nothing related to the project, only the CMake version.
My guess is some CMP changed in 3.25 which affects
cmake_dependent_option
.