Skip to content

Commit 414c6a5

Browse files
committed
Fix CI toolchain drift failures
The PR was failing before it reached the PyYAML regression tests in two places: the cp38 manylinux1 job installed latest cibuildwheel, whose 4.x series no longer supports Python 3.8, and the Windows arm64 libyaml build used a newer CMake that rejects libyaml 0.2.5's old minimum policy version. Pin the cp38 manylinux1 job to cibuildwheel<4, pass CMAKE_POLICY_VERSION_MINIMUM=3.5 to the Windows libyaml configure step, and disable fail-fast for the Windows libyaml matrix so one architecture does not hide the others.
1 parent 5a04f7f commit 414c6a5

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ jobs:
145145
with:
146146
matrix_yaml: |
147147
include:
148-
- { platform: manylinux1, arch: x86_64, spec: cp38 }
148+
- { platform: manylinux1, arch: x86_64, spec: cp38, cibw_version: 'cibuildwheel<4' }
149149
- { platform: manylinux1, arch: x86_64, spec: cp39, omit: ${{ env.skip_ci_redundant_jobs }} }
150150
- { platform: manylinux2014, arch: x86_64, spec: cp310, omit: ${{ env.skip_ci_redundant_jobs }} }
151151
- { platform: manylinux2014, arch: x86_64, spec: cp311, omit: ${{ env.skip_ci_redundant_jobs }} }
@@ -450,6 +450,7 @@ jobs:
450450
name: libyaml windows ${{ matrix.arch }}
451451
runs-on: ${{ (matrix.arch == 'arm64') && 'windows-11-arm' || 'windows-2022' }}
452452
strategy:
453+
fail-fast: false
453454
matrix:
454455
include:
455456
- arch: x64
@@ -481,7 +482,7 @@ jobs:
481482
mkdir libyaml/build
482483
483484
pushd libyaml/build
484-
cmake.exe -G "Visual Studio 17 2022" -A ${{ matrix.arch }} -DYAML_STATIC_LIB_NAME=yaml ..
485+
cmake.exe -G "Visual Studio 17 2022" -A ${{ matrix.arch }} -DYAML_STATIC_LIB_NAME=yaml -DCMAKE_POLICY_VERSION_MINIMUM=3.5 ..
485486
cmake.exe --build . --config Release
486487
popd
487488

0 commit comments

Comments
 (0)