-
Notifications
You must be signed in to change notification settings - Fork 73
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
bump minimum cmake requirement to 3.16 #920
Conversation
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.
LGTM, thanks!
oh hm, I just realized Debian 11 ships with 3.13. Does it make any sense to just go to 3.13 instead? is there anything useful added between 3.14 and 3.16? I looked through the release notes and nothing huge stood out... |
Given Bookworm (12) releases soon and I feel Bullseye (11) user should expect they will need to either upgrade to 12 or add bullseye-backports to get CMake 3.16, I think it's acceptable to leave this as is. Additionally, it's fairly easy to build a deb package from CMake source. |
While we always try to support as many as platforms as possible by default, as we officially support only Ubuntu 20.04 and 22.04, it is OK for us to go to the lowest supported OS's cmake 3.16. You can call this out in the 4.0 release note along with the removal of ubuntu 18.04 announcement. |
Yeah, combined
makes it seem like not a major hindrance. |
With the removal of Ubuntu 18.04 support we're clear to bump the minimum cmake requirement to 3.16 (the version included in Ubuntu 20.04). Interest in C++20 is primary immediate driver.
The only other notable platform loss with this change is Debian 10 which, up until this change, could still build Leap completely with system provided packages (Debian 10 included boost 1.67 even). With Debian 12 shortly releasing this isn't much of a concern really.
PR includes a few obvious small changes to remove now unneeded
CMAKE_VERSION
checks; I certainly don't claim to have refactored everything that could have been refactored for newer cmake here in this PR.