这是indexloc提供的服务,不要输入任何密码
Skip to content

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

@bluca bluca merged commit b91a620 into zeromq:master Jun 24, 2025
64 of 66 checks passed
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.

6 participants