Skip to content

Conversation

@musicinmybrain
Copy link

CMake 4.0 drops support for CMake <3.5, so minimum versions need to be adjusted for forward compatibility. As long as we are touching the minimum version, let’s advance to 3.12, which can be considered the oldest “modern” CMake release, and which is available in all known supported Linux distributions today.

CMake 4.0 drops support for CMake <3.5, so minimum versions need to be
adjusted for forward compatibility. As long as we are touching the
minimum version, let’s advance to 3.12, which can be considered the
oldest “modern” CMake release, and which is available in all known
supported Linux distributions today.
@@ -1,4 +1,4 @@
cmake_minimum_required(VERSION 2.8.11)
cmake_minimum_required(VERSION 3.12)

Choose a reason for hiding this comment

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

Suggested change
cmake_minimum_required(VERSION 3.12)
cmake_minimum_required(VERSION 3.12...3.5)
Suggested change
cmake_minimum_required(VERSION 3.12)
cmake_minimum_required(VERSION 3.12)

Copy link
Author

Choose a reason for hiding this comment

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

I’m confused. The first suggestion seems to change this to a version range, but in the wrong direction (3.12...3.5 instead of 3.5...3.12), and then the second suggestion … changes it back?

If you do have a concrete reason to want to support such old CMake versions, 3.5...3.12 would be a reasonable alternative to 3.12.

Copy link

@nim65s nim65s left a comment

Choose a reason for hiding this comment

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

Thanks @musicinmybrain :)

Also ping @Mengna-Li : your suggestion does not seem right. Is this a typo ? And what is the rationale ?

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