Fix testing wheels?

halide · Sep 3, 2024 · 828f0ca · 828f0ca
1 parent 7baccd7
commit 828f0ca
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/.github/workflows/pip.yml b/.github/workflows/pip.yml
@@ -208,15 +208,16 @@ jobs:
       # Wheels
       ########################################################################
 
-      - name: Setup tmate session
-        uses: mxschmitt/action-tmate@v3
+      #- uses: mxschmitt/action-tmate@v3
 
       - name: Build wheels
         uses: pypa/cibuildwheel@v2.20.0
         env:
           CIBW_BUILD: "cp3*-${{ matrix.platform_tag }}"
           CIBW_SKIP: "cp3{5,6,7}*"
-          CIBW_ENVIRONMENT_MACOS: "CMAKE_PREFIX_PATH='${{ github.workspace }}/opt'"
+          CIBW_ENVIRONMENT_MACOS: >
+            CMAKE_PREFIX_PATH='${{ github.workspace }}/opt'
+            Python_ROOT_DIR=''
           CIBW_ENVIRONMENT_WINDOWS: >
             CMAKE_GENERATOR=Ninja
             CMAKE_PREFIX_PATH='${{ github.workspace }}\opt'
@@ -225,6 +226,12 @@ jobs:
             cmake -G Ninja -S {project}/python_bindings/apps -B build -DCMAKE_BUILD_TYPE=Release &&
             cmake --build build &&
             ctest --test-dir build --output-on-failure
+          # Windows puts the Python interpreter in /Scripts, rather than /bin, which CMake doesn't understand.
+          CIBW_TEST_COMMAND_WINDOWS: >
+            cmake -G Ninja -S {project}/python_bindings/apps -B build -DCMAKE_BUILD_TYPE=Release
+            -DHalide_ROOT_DIR=$(./venv/Scripts/python -c 'import sysconfig; print(sysconfig.get_path("platlib"))') &&
+            cmake --build build &&
+            ctest --test-dir build --output-on-failure
 
       - uses: actions/upload-artifact@v4
         with: