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

update our CMake supported versions to ...3.31 #8727

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

lrineau
Copy link
Member

@lrineau lrineau commented Feb 4, 2025

Summary of Changes

Update our CMake supported versions to ...3.31. That will suppress the warnings about CMP0167 (from CMake 3.30):

CMake Warning (dev) at cmake/modules/display-third-party-libs-versions.cmake:37 (find_package):
  Policy CMP0167 is not set: The FindBoost module is removed.  Run "cmake
  --help-policy CMP0167" for policy details.  Use the cmake_policy command to
  set the policy and suppress this warning.

Release Management

  • Affected package(s): all cmake scripts
  • License and copyright ownership: maintenance by GF

That will suppress the warnings about `CMP0167` (from CMake 3.30):

```
CMake Warning (dev) at cmake/modules/display-third-party-libs-versions.cmake:37 (find_package):
  Policy CMP0167 is not set: The FindBoost module is removed.  Run "cmake
  --help-policy CMP0167" for policy details.  Use the cmake_policy command to
  set the policy and suppress this warning.
```
@lrineau lrineau added CMake scripts Warnings For an issue about warnings, or a pull-request that fixes warnings labels Feb 4, 2025
@lrineau lrineau added this to the 6.1-beta milestone Feb 4, 2025
@sloriot
Copy link
Member

sloriot commented Feb 7, 2025

Successfully tested in CGAL-6.1-Ic-81

@sloriot sloriot merged commit e871025 into CGAL:master Feb 7, 2025
8 checks passed
@sloriot sloriot deleted the CGAL-cmake-3.31 branch February 7, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake scripts Tested Warnings For an issue about warnings, or a pull-request that fixes warnings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants