seems good now, so build all

Author: aaronvg

.github/workflows/build-python-release.reusable.yaml

@@ -21,40 +21,40 @@ jobs:
       fail-fast: false
       matrix:
         _:
-          # - target: x86_64-unknown-linux-gnu
-          #   runs_on: ubuntu-latest
-          #   manylinux: 2_17
-
-          # - target: aarch64-unknown-linux-gnu
-          #   runs_on: ubuntu-latest
-          #   manylinux: 2_24
-          #   # I'm not sure if this actually works; I can't 'pip install ./local-path.whl'
-          #   # from inside a container based off quay.io/pypa/manylinux2014_aarch64
-          #   # see https://github.com/astral-sh/uv/issues/3439#issuecomment-2110448346
-          #   # manylinux: 2_28
-          #   # env:
-          #   #   # Workaround ring 0.17 build issue
-          #   #   # see https://github.com/briansmith/ring/issues/1728
-          #   #   CFLAGS_aarch64_unknown_linux_gnu: "-D__ARM_ARCH=8"
-
-          # - target: x86_64-unknown-linux-musl
-          #   runs_on: ubuntu-latest
-          #   # see https://github.com/astral-sh/uv/blob/9bb55c4ac0582e05d1a7a5bbd99cc7b2c82f1847/.github/workflows/build-binaries.yml#L594
-          #   manylinux: musllinux_1_1
-
-          # - target: aarch64-unknown-linux-musl
-          #   runs_on: ubuntu-latest
-          #   # see https://github.com/astral-sh/uv/blob/9bb55c4ac0582e05d1a7a5bbd99cc7b2c82f1847/.github/workflows/build-binaries.yml#L594
-          #   manylinux: musllinux_1_1
-
-          # - target: x86_64-apple-darwin
-          #   runs_on: macos-latest
-
-          # - target: aarch64-apple-darwin
-          #   runs_on: macos-latest
-
-          # - target: x86_64-pc-windows-msvc
-          #   runs_on: windows-latest
+          - target: x86_64-unknown-linux-gnu
+            runs_on: ubuntu-latest
+            manylinux: 2_17
+
+          - target: aarch64-unknown-linux-gnu
+            runs_on: ubuntu-latest
+            manylinux: 2_24
+            # I'm not sure if this actually works; I can't 'pip install ./local-path.whl'
+            # from inside a container based off quay.io/pypa/manylinux2014_aarch64
+            # see https://github.com/astral-sh/uv/issues/3439#issuecomment-2110448346
+            # manylinux: 2_28
+            # env:
+            #   # Workaround ring 0.17 build issue
+            #   # see https://github.com/briansmith/ring/issues/1728
+            #   CFLAGS_aarch64_unknown_linux_gnu: "-D__ARM_ARCH=8"
+
+          - target: x86_64-unknown-linux-musl
+            runs_on: ubuntu-latest
+            # see https://github.com/astral-sh/uv/blob/9bb55c4ac0582e05d1a7a5bbd99cc7b2c82f1847/.github/workflows/build-binaries.yml#L594
+            manylinux: musllinux_1_1
+
+          - target: aarch64-unknown-linux-musl
+            runs_on: ubuntu-latest
+            # see https://github.com/astral-sh/uv/blob/9bb55c4ac0582e05d1a7a5bbd99cc7b2c82f1847/.github/workflows/build-binaries.yml#L594
+            manylinux: musllinux_1_1
+
+          - target: x86_64-apple-darwin
+            runs_on: macos-latest
+
+          - target: aarch64-apple-darwin
+            runs_on: macos-latest
+
+          - target: x86_64-pc-windows-msvc
+            runs_on: windows-latest
 
           - target: aarch64-pc-windows-msvc
             runs_on: windows-11-arm
@@ -95,7 +95,7 @@ jobs:
         with:
           python-version: ${{ matrix._.target == 'aarch64-pc-windows-msvc' && '3.11' || '3.8' }}
           architecture: ${{ matrix._.architecture }}
-        id: python311
+        id: python-setup
 
       - name: Build wheels
         uses: PyO3/maturin-action@v1
@@ -107,7 +107,7 @@ jobs:
           command: build
           # building in engine/ ensures that we pick up .cargo/config.toml
           working-directory: engine
-          args: --release --out language_client_python/dist --manifest-path language_client_python/Cargo.toml --interpreter ${{ steps.python311.outputs.python-path }}
+          args: --release --out language_client_python/dist --manifest-path language_client_python/Cargo.toml ${{ matrix._.target == 'aarch64-pc-windows-msvc' && format('--interpreter {0}', steps.python-setup.outputs.python-path) || '' }}
           manylinux: ${{ matrix._.manylinux }}
           before-script-linux: |
             if command -v yum &> /dev/null; then