Move musl wheel to separate job

Chia-Network · Jan 25, 2024 · 7ddab60 · 7ddab60
1 parent 818d9e3
commit 7ddab60
Showing 1 changed file with 20 additions and 13 deletions.
diff --git a/.github/workflows/build-test.yml b/.github/workflows/build-test.yml
@@ -24,7 +24,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [macos-latest, ubuntu-latest, windows-latest]
-        python: [3.8, 3.9, "3.10", 3.11]
+        python: ["3.8", "3.9", "3.10", "3.11"]
 
     steps:
       - uses: actions/checkout@v3
@@ -61,12 +61,13 @@ jobs:
           . ./activate
           python -m pip install --upgrade pip
 
-      - name: Build MacOs with maturin on Python ${{ matrix.python }}
-        if: startsWith(matrix.os, 'macos')
+      - name: Build ${{ matrix.os }} with maturin on Python ${{ matrix.python }}
+        if: startsWith(matrix.os, 'macos') || startsWith(matrix.os, 'windows')
         env:
+          # this is ignored on Windows
           MACOSX_DEPLOYMENT_TARGET: "11.0"
         run: |
-
+          . ./activate
           maturin build -i python --release --strip
 
       - name: Build Linux in manylinux2014 with maturin on Python ${{ matrix.python }}
@@ -78,12 +79,6 @@ jobs:
           # Ensure an empty .cargo-lock file exists.
           touch target/release/.cargo-lock
 
-      - name: Build Windows with maturin on Python ${{ matrix.python }}
-        if: startsWith(matrix.os, 'windows')
-        run: |
-          . ./activate
-          maturin build -i python --release --strip
-
       - name: Install clvm_tools_rs wheel
         run: |
           . ./activate
@@ -212,12 +207,23 @@ jobs:
           cd clvm_tools
           pytest tests
 
+      - name: Upload artifacts
+        uses: actions/upload-artifact@v3
+        with:
+          name: wheels
+          path: ./target/wheels/
+
+  build_musl_wheel:
+    name: Build musl wheel on alpine docker on ubuntu
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
       - name: Build alpine wheel via docker
-        if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.python, '3.8')
         run: |
           cd resources/alpine && docker build -t clvm-tools-rs-alpine .
           docker run -v ${GITHUB_WORKSPACE}:/root/clvm_tools_rs -t clvm-tools-rs-alpine sh /root/build-alpine.sh
-
       - name: Upload artifacts
         uses: actions/upload-artifact@v3
         with:
@@ -338,7 +344,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [macos-latest, ubuntu-latest, windows-latest]
+        # should also run on windows, but need to fix path issues
+        os: [macos-latest, ubuntu-latest]
     name: Unit tests
     steps:
       - uses: actions/checkout@v3