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

CMake: CMP0075 requires CMake >= 3.12. #2218

Merged
merged 1 commit into from
Feb 10, 2022
Merged

CMake: CMP0075 requires CMake >= 3.12. #2218

merged 1 commit into from
Feb 10, 2022

Conversation

scivision
Copy link
Contributor

@scivision scivision commented Feb 7, 2022

The presence of cmake_policy(CMP0075 requires CMake >=3.12, and
errors with older CMake. Since older versions of CMake than 3.12
are already broken with NetCDF and uncommon nowadays, it may be
better to explicitly require 3.12 or newer.

This also enables hinting external libraries such as HDF5 via variable HDF5_ROOT, which is very handy
when using NetCDF from an ExternalProject in CMake.

This also suppresses several nuisance warnings about unexpected behavior when using a modern CMake version.

The presence of cmake_policy(CMP0075 required CMake >=3.12, and
errors with older CMake. Since older versions of CMake than 3.12
are already broken with NetCDF and uncommon nowadays, it may be
better to explicitly require 3.12 or newer.
@scivision scivision requested a review from WardF as a code owner February 7, 2022 16:17
@CLAassistant
Copy link

CLAassistant commented Feb 7, 2022

CLA assistant check
All committers have signed the CLA.

@WardF
Copy link
Member

WardF commented Feb 10, 2022

The error we're seeing on the OSX test is known and frustrating; sorting out what to do about it.

@WardF
Copy link
Member

WardF commented Feb 10, 2022

The OSX failure is for a known and unrelated issue, so merging anyways. Thank you @scivision

@WardF WardF merged commit 53fdee6 into Unidata:main Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants