Skip to content

Conversation

lgritz
Copy link
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
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