Skip to content

build: Support for CMake 4.0#1960

Merged
lgritz merged 1 commit intoAcademySoftwareFoundation:mainfrom
lgritz:lg-cmake4
Apr 3, 2025
Merged

build: Support for CMake 4.0#1960
lgritz merged 1 commit intoAcademySoftwareFoundation:mainfrom
lgritz:lg-cmake4

Conversation

@lgritz
Copy link
Copy Markdown
Collaborator

@lgritz lgritz commented Apr 1, 2025

The tricky part is that several dependencies break because of cmake 4.0 dropping support for strict compatbility with cmake < 3.5. Have to resort to some trickery with CMAKE_POLICY_VERSION_MINIMUM overriding what those packages declare via their own cmake_minimum_required(), and hope it works.

The tricky part is that several dependencies break because of cmake
4.0 dropping support for strict compatbility with cmake < 3.5. Have to
resort to some trickery with CMAKE_POLICY_VERSION_MINIMUM overriding
what those packages declare via their own cmake_minimum_required(),
and hope it works.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
@lgritz
Copy link
Copy Markdown
Collaborator Author

lgritz commented Apr 3, 2025

Merging as this is desperately needed to fix CI.

There are still two tests failing, but it's for a different reason (llvm 20 issues) and I will address those separately.

@lgritz lgritz merged commit 323c1c1 into AcademySoftwareFoundation:main Apr 3, 2025
24 of 26 checks passed
@lgritz lgritz deleted the lg-cmake4 branch April 3, 2025 04:46
lgritz added a commit to lgritz/OpenShadingLanguage that referenced this pull request Apr 3, 2025
The tricky part is that several dependencies break because of cmake
4.0 dropping support for strict compatbility with cmake < 3.5. Have to
resort to some trickery with CMAKE_POLICY_VERSION_MINIMUM overriding
what those packages declare via their own cmake_minimum_required(),
and hope it works.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
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.

1 participant