Skip to content

Commit

Permalink
Github runner: Upgrade LLVM to LLVM 16
Browse files Browse the repository at this point in the history
Recent github runner config changes for Windows breaks use of LLVM 15
and does not install LLVM 16 in the image.

This change adds an LLVM upgrade step that should pull in LLVM 16. This
follows scikit-image/scikit-image#7109.

Bug: actions/runner-images#812
  • Loading branch information
ohodson committed Aug 29, 2023
1 parent 7d16d86 commit 0de0d83
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
uses: actions/cache@v3
with:
path: ~/bazel-disk-cache
key: bazel-disk-cache-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('.bazelversion', '.bazelrc', 'WORKSPACE') }}
key: bazel-disk-cache-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('.bazelversion', '.bazelrc', 'WORKSPACE', '.github/workflows/test.yml') }}
# Use an older cache key, if available.
restore-keys: |
bazel-disk-cache-${{ runner.os }}-${{ runner.arch }}-
Expand Down Expand Up @@ -57,8 +57,10 @@ jobs:
# bazel generates huge amounts of debug information on Windows which slows down the build
# and takes up an excessive amount of cache space.
run: |
bazelisk clean --expunge
[System.IO.File]::WriteAllLines((Join-Path -Path $env:USERPROFILE -ChildPath '.bazelrc'), 'startup --output_user_root=C:/tmp')
[System.IO.File]::WriteAllLines((Join-Path -Path $env:USERPROFILE -ChildPath '.bazelrc'), 'build:windows --config=windows_no_dbg')
choco upgrade llvm
- name: Bazel build
# timestamps are no longer being added here, the GitHub logs include timestamps (Use
# 'Show timestamps' on the web interface)
Expand Down

0 comments on commit 0de0d83

Please sign in to comment.