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

Change cmake minimum version to 3.5 #156

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Conversation

szekipepe
Copy link
Contributor

Change cmake minimum version to 3.5 to work after 3.27 versions of cmake.

log with cmake 3.27.6:
Compatibility with CMake < 3.5 will be removed from a future version of CMake.

Change cmake minimum version to 3.5 to work after 3.27 versions of cmake.

log with cmake 3.27.6:
Compatibility with CMake < 3.5 will be removed from a future version of CMake.
@szekipepe
Copy link
Contributor Author

I just thinking... maybe we can use cmake minimum version as this:

cmake_minimum_required(VERSION 3.1...3.5)

Would that be better?

@neithernut
Copy link

Disclaimer: not a maintainer.

Just had a look out of curiosity, and CMake 3.5 was apparently released in March 2016, more than seven years ago1. I doubt anyone building new(er) software would use an older version, and maintainers of legacy code still using such an old version of CMake are likely ok with not upgrading some dependencies. I think it's ok to just bump the CMake version like you did.

Footnotes

  1. I"m honestly pleasantly surprised they still have the old release pages up and usable on their current website.

@foonathan foonathan merged commit 2608b7e into foonathan:main Oct 12, 2023
7 checks passed
@foonathan
Copy link
Owner

Yeah, I think requiring 3.5 is fine.

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.

3 participants