Skip to content

Commit 9310aa5

Browse files
author
Alexander Refsum Jensenius
committed
Improve CI robustness for macOS builds
- Use brew install ffmpeg for macOS instead of setup-ffmpeg action - Install core dependencies (numpy, scipy, matplotlib, opencv-python) first - Add basic import test to verify installation - Separate FFmpeg installation by OS for better reliability - Should resolve Python 3.10 macOS build failures
1 parent d841967 commit 9310aa5

File tree

1 file changed

+20
-6
lines changed

1 file changed

+20
-6
lines changed

.github/workflows/ci.yml

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,30 @@ jobs:
2222
steps:
2323
- uses: actions/checkout@v4
2424

25-
- name: Install FFmpeg and FFprobe
26-
uses: FedericoCarboni/setup-ffmpeg@v3
27-
id: setup-ffmpeg
28-
2925
- name: Set up Python ${{ matrix.python-version }}
3026
uses: actions/setup-python@v5
3127
with:
3228
python-version: ${{ matrix.python-version }}
3329

34-
- name: Install dependencies
30+
- name: Install system dependencies on macOS
31+
if: runner.os == 'macOS'
32+
run: |
33+
brew install ffmpeg
34+
35+
- name: Install FFmpeg on Ubuntu/Windows
36+
if: runner.os != 'macOS'
37+
uses: FedericoCarboni/setup-ffmpeg@v3
38+
id: setup-ffmpeg
39+
40+
- name: Install Python dependencies
3541
run: |
3642
python -m pip install --upgrade pip
37-
python -m pip install musicalgestures
43+
python -m pip install numpy scipy matplotlib opencv-python
44+
45+
- name: Install musicalgestures
46+
run: |
47+
python -m pip install musicalgestures
48+
49+
- name: Test basic import
50+
run: |
51+
python -c "import musicalgestures; print('MGT-python import successful')"

0 commit comments

Comments
 (0)