Skip to content

Commit c803525

Browse files
authored
Merge pull request DeusData#2028 from DeusData/ci/pin-llvm22-lsan-macos
ci(lsan-macos): pin the Homebrew LLVM toolchain to llvm@22
2 parents 44fa3ae + 1d38be3 commit c803525

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

‎.github/workflows/_test.yml‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -311,8 +311,14 @@ jobs:
311311
steps:
312312
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
313313

314+
# Pinned: the unversioned `llvm` formula tracks Homebrew stable, so the
315+
# leg's compiler changed under us when stable moved 22 -> 23 (a new
316+
# -Wunused-but-set-global under -Werror broke the build on whichever
317+
# runner drew the 23 bottle, and passed on the ones that drew 22). A
318+
# test verdict must be a function of the code, not of the image snapshot;
319+
# moving to a newer major is a deliberate one-line bump here.
314320
- name: Install LSan-capable toolchain
315-
run: brew install llvm ccache
321+
run: brew install llvm@22 ccache
316322

317323
- name: Compiler cache (content-verified, restore)
318324
uses: actions/cache/restore@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6.1.0
@@ -327,7 +333,7 @@ jobs:
327333
env:
328334
ASAN_OPTIONS: "detect_leaks=1:halt_on_error=1"
329335
run: |
330-
LLVM_PREFIX="$(brew --prefix llvm)"
336+
LLVM_PREFIX="$(brew --prefix llvm@22)"
331337
scripts/test.sh CC="$LLVM_PREFIX/bin/clang" CXX="$LLVM_PREFIX/bin/clang++"
332338
333339
- name: Compiler cache (save)

0 commit comments

Comments
 (0)