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

fixed #13343 - Bump cmake_minimum_required() to 3.13 #7213

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

rouault
Copy link
Contributor

@rouault rouault commented Jan 12, 2025

and do associated cleanups.

Fixes #13343

Recent CMake versions emit:

CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 3.10 will be removed from a future version of
  CMake.
  Update the VERSION argument <min> value.  Or, use the <min>...<max> syntax
  to tell CMake that the project requires at least <min> but has been updated
  to work with policies introduced by <max> or earlier.

@firewave
Copy link
Collaborator

@firewave firewave self-requested a review January 13, 2025 12:40
Copy link
Collaborator

@firewave firewave left a comment

Choose a reason for hiding this comment

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

I checked https://pkgs.org and the notes in #3596 and bumping it further up to 3.13 should be no problem. So the additional cmake_minimum_required() is no longer required.

Please also update readme.md. And there is several CMAKE_VERSION checks which can be removed.

@rouault rouault changed the title Bump cmake_minimum_required() to 3.10 Bump cmake_minimum_required() to 3.13 Jan 13, 2025
and do associated cleanups.

Fixes #13343

Recent CMake versions emit:
```
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 3.10 will be removed from a future version of
  CMake.
  Update the VERSION argument <min> value.  Or, use the <min>...<max> syntax
  to tell CMake that the project requires at least <min> but has been updated
  to work with policies introduced by <max> or earlier.
```
@rouault
Copy link
Contributor Author

rouault commented Jan 13, 2025

bumping it further up to 3.13 should be no problem. So the additional cmake_minimum_required() is no longer required.

Please also update readme.md. And there is several CMAKE_VERSION checks which can be removed.

done

@firewave firewave changed the title Bump cmake_minimum_required() to 3.13 fixed #13343 - Bump cmake_minimum_required() to 3.13 Jan 13, 2025
@firewave firewave changed the title fixed #13343 - Bump cmake_minimum_required() to 3.13 fixed #13343 - Bump cmake_minimum_required() to 3.13 Jan 13, 2025
Copy link
Collaborator

@firewave firewave left a comment

Choose a reason for hiding this comment

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

@danmar you okay with this?

Copy link
Owner

@danmar danmar left a comment

Choose a reason for hiding this comment

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

of course :-)

@danmar danmar merged commit 54a9ba4 into danmar:main Jan 23, 2025
60 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.

3 participants