-
-
Notifications
You must be signed in to change notification settings - Fork 55
cmake: cmake_minimum_required 3.10 (3.9 is deprecated for cmake 3.31 onwards) #199
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
Conversation
Why is that? What is the limiting factor for earlier versions? 3.10 Seems reasonable as that is what is available in Ubuntu 18 repos |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #199 +/- ##
===========================================
- Coverage 99.10% 97.73% -1.37%
===========================================
Files 34 34
Lines 3591 3052 -539
===========================================
- Hits 3559 2983 -576
- Misses 32 69 +37 🚀 New features to boost your workflow:
|
Good question. I dug some more. Cobalt (cmake-3.10.0 fails, cmake-3.12.0 succeeds)
Heap (cmake-3.18.0 fails, cmake-3.19.0 succeeds)
cmake 3.19.0 (Nov 2020) does also work monolithically. Are you aware of some particular cmake baseline version that is supposed to be supported? |
Ok so Cobalt does require 3.12, Heap does have a bug in its CMakeLists which likely will be fixed in the next version.
That is only due to CMake 3.19 ignoring the bug mentioned above.
I'm not aware of that as there is no Boost-wide rule to support a specific minimum version.
To still allow 3.9 in general for Nowide I made a different PR that also fixes you issue. Note that CMake build support is still experimental so some issues might still need to be fixed |
To clarify. This PR #199 emits no warning for cmake-3.31.0. The alternative PR #200 emits the warning:
My goal here is to resolve the cmake deprecation warning. |
Thanks for testing! |
Confirming that. Thanks for the fix. |
The current version of make is 4.1.1
It's complaining that cmake 3.9 policy is deprecated since 3.31
This change brings the minimum cmake requirement to 3.10 (Nov 2017).
Boost more broadly needs cmake 3.20 (Apr 2021) or newer to build monolithically.