Skip to content

Commit

Permalink
🔧 (CI): Fix missing python3.10 in Mac CI (#48)
Browse files Browse the repository at this point in the history
* fix CI

* fix CI

* fix release CI
  • Loading branch information
Isotr0py authored Jun 15, 2024
1 parent fad4b9c commit 439403d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 7 deletions.
16 changes: 10 additions & 6 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -118,18 +118,22 @@ jobs:
path: dist

macos:
runs-on: macos-latest
runs-on: ${{ matrix.platform.runner }}
strategy:
matrix:
target: [x86_64, aarch64]
platform:
- runner: macos-latest
target: x86_64
- runner: macos-14
target: aarch64
steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: actions/setup-python@v5
with:
architecture: ${{ matrix.target }}
python-version: '3.10'

- name: Check dependencys
run: |
Expand All @@ -146,16 +150,16 @@ jobs:
# from homebrew
DEP_JXL_LIB: /opt/homebrew/Cellar/jpeg-xl/0.10.2/lib
DEP_BROTLI_LIB: /opt/homebrew/Cellar/brotli/1.1.0/lib
DEP_HWY_LIB: /opt/homebrew/Cellar/highway/1.1.0/lib
DEP_HWY_LIB: /opt/homebrew/Cellar/highway/1.2.0/lib
with:
target: ${{ matrix.target }}
target: ${{ matrix.platform.target }}
args: --release --out dist --find-interpreter --features dynamic
sccache: 'true'

- name: Upload wheels
uses: actions/upload-artifact@v4
with:
name: wheels-macos-${{ matrix.target }}
name: wheels-macos-${{ matrix.platform.target }}
path: dist

sdist:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
brew install jpeg-xl
export DEP_JXL_LIB=/opt/homebrew/Cellar/jpeg-xl/0.10.2/lib
export DEP_BROTLI_LIB=/opt/homebrew/Cellar/brotli/1.1.0/lib
export DEP_HWY_LIB=/opt/homebrew/Cellar/highway/1.1.0/lib
export DEP_HWY_LIB=/opt/homebrew/Cellar/highway/1.2.0/lib
source venv/bin/activate
pip install maturin
maturin develop --features dynamic
Expand Down

0 comments on commit 439403d

Please sign in to comment.