Skip to content

Commit a9dbb42

Browse files
committed
ci(fp-stability): derive Verrou cache key from verrou.sh content (no hand-synced version)
The cache key hardcoded verrou-a58d434-valgrind-3.26.0, synced to the installer's pins only by a comment — if verrou.sh bumped but the key didn't, CI would restore the stale cached tree and silently never exercise the new version. Key off hashFiles('toolchain/bootstrap/verrou.sh') so any pin change (or edit) auto-busts the cache. Also dropped the version literals from the workflow's header comment; the pinned versions now live solely in verrou.sh.
1 parent d809997 commit a9dbb42

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

.github/workflows/fp-stability.yml

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,9 @@ name: FP Stability
2424
# On FAIL: verrou_dd_sym runs to identify the responsible function symbols.
2525
# Logs are uploaded as CI artifacts.
2626
#
27-
# Verrou (Valgrind 3.26.0 + edf-hpc/verrou@a58d434) is built once and cached.
28-
# Build takes ~20 min uncached; cached runs restore in ~30 s.
27+
# Verrou (the pinned Valgrind+Verrou pair; versions live in toolchain/bootstrap/verrou.sh)
28+
# is installed by fp-stability on first use and cached. The prebuilt download is seconds;
29+
# a cache miss with no prebuilt falls back to a ~20-min source build.
2930

3031
on:
3132
push:
@@ -68,8 +69,10 @@ jobs:
6869
uses: actions/cache@v4
6970
with:
7071
path: ~/.local/verrou
71-
# Keep these versions in sync with toolchain/bootstrap/verrou.sh (the installer).
72-
key: verrou-a58d434-valgrind-3.26.0-${{ runner.os }}
72+
# Key off the installer's content so any version bump (or other edit) in
73+
# verrou.sh auto-busts the cache and forces a fresh install — no hand-synced
74+
# version string to drift out of date.
75+
key: verrou-${{ hashFiles('toolchain/bootstrap/verrou.sh') }}-${{ runner.os }}
7376

7477
- name: Install system dependencies
7578
run: |

0 commit comments

Comments
 (0)