wheel: assign CIBW_SKIP globally stable
authorPierre-Yves David <pierre-yves.david@octobus.net>
Sun, 10 Nov 2024 00:59:21 +0100
branchstable
changeset 52231 d0e728b0db0e
parent 52230 7a568296296e
child 52232 763817eca85e
wheel: assign CIBW_SKIP globally The other cibuildwheels (e.g. windows) will needs it too.
contrib/heptapod-ci.yml
--- a/contrib/heptapod-ci.yml	Sun Nov 10 22:30:02 2024 +0100
+++ b/contrib/heptapod-ci.yml	Sun Nov 10 00:59:21 2024 +0100
@@ -36,6 +36,9 @@
     # with shell runner, its content is not cleaned from one call to the next,
     # so plan for it.
     TMP_WORK_DIR: "${CI_PROJECT_DIR}/../.."
+    # we use CIBW_SKIP="pp*" to prevent the building of pypy wheel that are neither
+    # needed nor working.
+    CIBW_SKIP: "pp*"
 
 .all:
   # help changing all job at once when debugging
@@ -344,11 +347,11 @@
     needs:
       - build-c-wheel-macos
 
-# we use CIBW_SKIP="pp*" to prevent the building of pypy wheel that are neither
-# needed nor working.
+# We could use CIBW_BUILD="cp310-*" to only build the Python 3.10 wheel for now as
+# this is the only one we need to test. However testing that build work on all
+# version is useful and match what we do with Linux.
 #
-# We could use CIBW_BUILD="cp310-*" to only build the Python 3.10 wheel for now as
-# this is the only one we need to test. However testing that build work on all version is useful and match what we do with Linux.
+# CIBW_SKIP is set globally at the start of the file. See comment there.
 #
 # The weird directory structure match the one we use for Linux to deal with the
 # multiple jobs. (all this might be unnecessary)
@@ -361,7 +364,7 @@
       MERCURIAL_SETUP_FORCE_TRANSLATIONS: "1"
     script:
       - rm -rf tmp-wheels
-      - CIBW_SKIP="pp*" cibuildwheel --output-dir tmp-wheels/
+      - cibuildwheel --output-dir tmp-wheels/
       - for py_version in cp38-cp38 cp39-cp39 cp310-cp310 cp311-cp311 cp312-cp312 cp313-cp313; do
           mkdir -p wheels/c/$py_version/;
           mv tmp-wheels/*$py_version*.whl wheels/c/$py_version/;