Skip to content

Commit

Permalink
ci: Fix llvm detection on macOS CI
Browse files Browse the repository at this point in the history
LLVM is keg-only now, so we need to add CPPFLAGS / LDFLAGS to the keg
subdir inside /usr/local
  • Loading branch information
nirbheek committed Oct 1, 2022
1 parent 9fd5eb6 commit f80e153
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@ jobs:
python -m pip install pytest pytest-xdist pytest-subtests jsonschema coverage codecov
- run: brew install pkg-config ninja llvm qt@5
- env:
CPPFLAGS: "-I/usr/local/include"
LDFLAGS: "-L/usr/local/lib"
CPPFLAGS: "-I/usr/local/include -I/usr/local/opt/llvm/include"
LDFLAGS: "-L/usr/local/lib -L/usr/local/opt/llvm/lib"
MESON_CI_JOBNAME: unittests-appleclang
MESON_UNIT_TEST_BACKEND: ninja
# These cannot evaluate anything, so we cannot set PATH or SDKROOT here
Expand Down Expand Up @@ -75,8 +75,8 @@ jobs:
python3 -m pip install --upgrade pip
python3 -m pip install cython coverage codecov
- env:
CPPFLAGS: "-I/usr/local/include"
LDFLAGS: "-L/usr/local/lib"
CPPFLAGS: "-I/usr/local/include -I/usr/local/opt/llvm/include"
LDFLAGS: "-L/usr/local/lib -L/usr/local/opt/llvm/lib"
MESON_ARGS: --unity=${{ matrix.unity }}
CI: 1
# These cannot evaluate anything, so we cannot set PATH or SDKROOT here
Expand Down

0 comments on commit f80e153

Please sign in to comment.