Skip to content

Commit

Permalink
Fix env var casing, split matrix on python version for parallelism.
Browse files Browse the repository at this point in the history
  • Loading branch information
ScottTodd committed Sep 27, 2024
1 parent 0bddbc6 commit 19388d2
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions .github/workflows/build_packages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ on:

jobs:
build_packages:
name: "${{ matrix.build-family }} :: Build ${{ matrix.package }} Package"
name: "${{ matrix.build-family }} :: Build ${{ matrix.package }} Package for ${{ matrix.python-version }}"
runs-on: ${{ matrix.runs-on }}
strategy:
fail-fast: false
Expand All @@ -24,6 +24,11 @@ jobs:
- runs-on: ubuntu-24.04
build-family: linux-x86_64
package: shortfin
python-version: cp312-cp312
- runs-on: ubuntu-24.04
build-family: linux-x86_64
package: shortfin
python-version: cp313-cp313
# TODO(#130): macOS
# TODO(#130): Windows

Expand All @@ -35,11 +40,12 @@ jobs:
submodules: false


- name: Build shortfin (Linux x86_64)
- name: Build shortfin (Linux x86_64, ${{ matrix.python-version }})
if: "matrix.package == 'shortfin' && matrix.build-family == 'linux-x86_64'"
shell: bash
env:
output_dir: "${{ github.workspace }}/bindist"
OUTPUT_DIR: "${{ github.workspace }}/bindist"
OVERRIDE_PYTHON_VERSIONS: "${{ matrix.python-version }}"
run: |
[ -e ./bindist/* ] && rm ./bindist/*
./c/shortfin/build_tools/build_linux_package.sh
Expand Down

0 comments on commit 19388d2

Please sign in to comment.