Skip to content

set upper bound in cmake_minimum_required #4776

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

minrk
Copy link
Member

@minrk minrk commented Mar 20, 2025

setting an upper bound improves forward-compatibility as legacy version support is dropped. Without an upper bound, the minimum version is used as the policy version, which means that setting a version lower than 3.5 will result in warnings starting with cmake 3.27 and errors starting in 4.0 (release candidate out now). This change allows zeromq to build with cmake 4.

  • 3.5 compat is deprecated in 3.27 (2023), removed in 4.0 (2025)
  • 3.10 compat is deprecated in 3.31 (2024)

setting an upper bound improves forward-compatibility as legacy version support is dropped

- 3.5 compat is deprecated in 3.27 (2023), removed in 4.0 (2025)
- 3.10 compat is deprecated in 3.31 (2024)
@myd7349
Copy link
Contributor

myd7349 commented Apr 8, 2025

This will fix configuration error with CMake 4.0.

@mcloughlan
Copy link

getting this issue when using fetchcontent in a cmake file with version 4 as minimum as well

mcloughlan added a commit to RMIT-Competition-Rocketry/GCS that referenced this pull request Apr 15, 2025
Also found that fetchcontent will not work for libzmq until
zeromq/libzmq#4776 is resolved
@a4z
Copy link
Contributor

a4z commented May 11, 2025

found out too late that this pr exists, so I added #4789

which this pr would have already been merged, that would have spared me some time

@andoorve
Copy link

andoorve commented Jun 4, 2025

Hi, is there any blocker to this being merged?

@a4z
Copy link
Contributor

a4z commented Jun 4, 2025

Hi, is there any blocker to this being merged?

To me it seems, for some time, all PRs have failed.
For various reasons, either a specific specified clang version was not found that got updated in the base image, or new coverage versions introduce new warnings, which are triggered as errors, and/or simply flaky tests as it can happen in containers whith uncertain execution time

It's unfortunate, but nobody seems to investigate those topics, and I miss the time it would take to do that since my activity is atm not work releated :-(

@andoorve
Copy link

andoorve commented Jun 4, 2025

I see, it's indeed unfortunate. Looks like the last commit was 6 months ago.

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

Successfully merging this pull request may close these issues.

5 participants