refactor(ci): simplify process-updates cache cleanup #76

Merged

pat-s merged 1 commit from pr/cache-cleanup-magic-string into main

2026-06-12 13:41:54 +00:00

pat-s commented

2026-05-28 13:45:55 +00:00

Owner

Summary

The rm -rf /mnt/cache/... block in 14 process-updates-* files had two real issues:

/mnt/cache/pkgcache/R/pkgcache listed twice — harmless duplicate.
/mnt/cache/R-pkgs/00LOCK-pak/mnt/cache/R-pkgs/00LOCK-RPostgres — missing a space between two paths, so rm walks a nested nonsense path that never exists. The 00LOCK-RPostgres lock cleanup never actually ran.

Collapses to the same shape weekly-rebuild-missing-* already uses:

rm -rf /mnt/cache/R-pkgs/00LOCK-* /mnt/cache/R-pkgs/{bincraft,pkgcache} /mnt/cache/pkgcache/R/pkgcache

00LOCK-* covers every stale lock (pak, bincraft, RPostgres, r-pkg-binaries, …) without enumerating them, so adding a new locked package later doesn't need another YAML touch.

14 files, 1 line changed each.

## Summary The `rm -rf /mnt/cache/...` block in 14 `process-updates-*` files had two real issues: 1. `/mnt/cache/pkgcache/R/pkgcache` listed twice — harmless duplicate. 2. `/mnt/cache/R-pkgs/00LOCK-pak/mnt/cache/R-pkgs/00LOCK-RPostgres` — missing a space between two paths, so `rm` walks a nested nonsense path that never exists. The `00LOCK-RPostgres` lock cleanup never actually ran. Collapses to the same shape `weekly-rebuild-missing-*` already uses: ``` rm -rf /mnt/cache/R-pkgs/00LOCK-* /mnt/cache/R-pkgs/{bincraft,pkgcache} /mnt/cache/pkgcache/R/pkgcache ``` `00LOCK-*` covers every stale lock (pak, bincraft, RPostgres, r-pkg-binaries, …) without enumerating them, so adding a new locked package later doesn't need another YAML touch. 14 files, 1 line changed each.

pat-s added 1 commit

2026-05-28 13:45:55 +00:00

refactor(ci): simplify process-updates cache cleanup ae7aee9086

The cleanup string repeated across 14 process-updates-* files had:
- /mnt/cache/pkgcache/R/pkgcache listed twice (harmless duplicate)
- /mnt/cache/R-pkgs/00LOCK-pak/mnt/cache/R-pkgs/00LOCK-RPostgres
  (missing space, ends up rm'ing a nonsense nested path)

Collapse to the same form weekly-rebuild-missing-* already uses:

    rm -rf /mnt/cache/R-pkgs/00LOCK-* \
      /mnt/cache/R-pkgs/{bincraft,pkgcache} \
      /mnt/cache/pkgcache/R/pkgcache

00LOCK-* covers every stale lock (pak, bincraft, RPostgres,
r-pkg-binaries, ...) without enumerating them.

pat-s force-pushed pr/cache-cleanup-magic-string from ae7aee9086 to 5d5df70b81

2026-06-12 13:38:19 +00:00

Compare

pat-s merged commit 4242e63f30 into main

2026-06-12 13:41:54 +00:00

pat-s deleted branch pr/cache-cleanup-magic-string

2026-06-12 13:41:54 +00:00

pat-s referenced this pull request from a commit

2026-06-12 13:41:54 +00:00

refactor(ci): simplify process-updates cache cleanup (#76)

pat-s referenced this pull request from a commit

2026-06-13 20:00:01 +00:00