Skip to content

Commit 358386e

Browse files
committed
Fix ccache restore key for check-macos-12-cmake-clang
We must not pick up a cache generated by the non-Glucose job, which has the same prefix. Doing so would result in a cache that mixes different configurations, and ccache may end up pruning the wrong ones. While at it, also insert hyphens to make cache entry names easier to read.
1 parent 3f578e1 commit 358386e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/pull-request-checks.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -521,10 +521,10 @@ jobs:
521521
uses: actions/cache@v3
522522
with:
523523
path: .ccache
524-
key: ${{ runner.os }}-Release-Glucose${{ github.ref }}-${{ github.sha }}-PR
524+
key: ${{ runner.os }}-Release-Glucose-${{ github.ref }}-${{ github.sha }}-PR
525525
restore-keys: |
526-
${{ runner.os }}-Release-Glucose${{ github.ref }}
527-
${{ runner.os }}-Release
526+
${{ runner.os }}-Release-Glucose-${{ github.ref }}
527+
${{ runner.os }}-Release-Glucose
528528
- name: ccache environment
529529
run: |
530530
echo "CCACHE_BASEDIR=$PWD" >> $GITHUB_ENV

0 commit comments

Comments
 (0)