Skip to content

Commit aae1776

Browse files
committed
Manylinux temporary fix: Downgrade CMake to 3.x via pipx in manylinux
This avoids CMake errors introduced with CMake 4.0 related to 3rd party dependencies with cmake_minimum_required(3.5) or older This is needed for rapdijson at configure time, and for visualiastion builds the el8 installation of DevIL from source via CMake. See #1276
1 parent 6de71ca commit aae1776

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

.github/workflows/Draft-Release.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -376,6 +376,11 @@ jobs:
376376
cuda: ${{ env.CUDA }}
377377
run: .github/scripts/install_cuda_el8.sh
378378

379+
# Downgrade CMake for DeVIL and rapidjson. See https://github.com/FLAMEGPU/FLAMEGPU2/issues/1276
380+
- name: Install CMake < 4.0, overwriting the pipx installed version from manylinux
381+
run: |
382+
pipx install --global --force 'cmake<4.0'
383+
379384
- name: Install Visualisation Dependencies (EL 8)
380385
if: ${{ env.VISUALISATION == 'ON' }}
381386
run: |

.github/workflows/Manylinux_2_28.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,11 @@ jobs:
105105
cuda: ${{ env.CUDA }}
106106
run: .github/scripts/install_cuda_el8.sh
107107

108+
# Downgrade CMake for DeVIL and rapidjson. See https://github.com/FLAMEGPU/FLAMEGPU2/issues/1276
109+
- name: Install CMake < 4.0, overwriting the pipx installed version from manylinux
110+
run: |
111+
pipx install --global --force 'cmake<4.0'
112+
108113
- name: Install Visualisation Dependencies (EL 8)
109114
if: ${{ env.VISUALISATION == 'ON' }}
110115
run: |

0 commit comments

Comments
 (0)