Skip to content

Commit 7aed0a8

Browse files
authored
Move Install homebrew dependencies (#19403)
* move Install homebrew dependencies * minor changes
1 parent bd7ad6a commit 7aed0a8

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

.github/workflows/osx-tests.yml

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,18 @@ jobs:
147147
xcrun --sdk macosx --show-sdk-version
148148
clang --version
149149
150+
# Install system dependencies BEFORE setting up the matrix Python.
151+
# This prevents Emscripten (which requires Python 3.10+) from crashing
152+
# when trying to use Python 3.8 from the matrix.
153+
- name: Install homebrew dependencies
154+
run: |
155+
brew update
156+
brew install xcodegen make libtool zlib autoconf automake ninja emscripten
157+
export PATH=${HOME}/Applications/CMake/3.15.7/bin:$PATH:/usr/local/bin:/opt/homebrew/bin:/usr/bin:/bin:/usr/sbin:/sbin
158+
emcc --version
159+
cmake --version
160+
bazel --version
161+
150162
- name: Set up Python ${{ matrix.python-version }}
151163
uses: actions/setup-python@v5
152164
with:
@@ -160,15 +172,6 @@ jobs:
160172
pip install -r conans/requirements_dev.txt
161173
pip install meson
162174
163-
- name: Install homebrew dependencies
164-
run: |
165-
brew update
166-
brew install xcodegen make libtool zlib autoconf automake ninja emscripten
167-
export PATH=${HOME}/Applications/CMake/3.15.7/bin:$PATH:/usr/local/bin:/opt/homebrew/bin:/usr/bin:/bin:/usr/sbin:/sbin
168-
emcc --version
169-
cmake --version
170-
bazel --version
171-
172175
- name: Run tests
173176
uses: ./.github/actions/test-coverage
174177
with:

0 commit comments

Comments
 (0)