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

bump minimum cmake requirement to 3.16 #920

Merged
merged 4 commits into from
Apr 1, 2023
Merged

bump minimum cmake requirement to 3.16 #920

merged 4 commits into from
Apr 1, 2023

Conversation

spoonincode
Copy link
Member

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.

@ScottBailey ScottBailey self-requested a review March 29, 2023 21:19
Copy link
Contributor

@ScottBailey ScottBailey left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@spoonincode
Copy link
Member Author

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...

@ScottBailey
Copy link
Contributor

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.

@linh2931
Copy link
Member

linh2931 commented Apr 1, 2023

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.

@spoonincode
Copy link
Member Author

Yeah, combined

  • Debian isn't a primary support target
  • 3.16 is available in bullseye-backports, and
  • By the the time 5.0 is released Debian 12 will have been out for months

makes it seem like not a major hindrance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants