[llvm] [GitHub] Remove redundant cache key prefix (PR #76914)

Craig Hesling via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 3 22:28:30 PST 2024


https://github.com/linux4life798 created https://github.com/llvm/llvm-project/pull/76914

Remove the redundant sccache cache key prefix.
This prefix is already added by the ccache action, which results in cache keys like "sccache-sccache-ubuntu-...".

See the following source lines as proof:
https://github.com/hendrikmuhs/ccache-action/blob/2a51777f6f64b7b7bea213601acba8f5f4fdbe03/src/restore.ts#L22-L23

>From 5ad94400ff9d9b8c9d5e937239ff2dc4a9f6e51e Mon Sep 17 00:00:00 2001
From: Craig Hesling <craig at hesling.com>
Date: Thu, 4 Jan 2024 01:23:30 -0500
Subject: [PATCH] [GitHub] Remove redundant cache key prefix

Remove the redundant sccache cache key prefix.
This prefix is already added by the ccache action,
which results in cache keys like "sccache-sccache-ubuntu-...".

See the following source lines as proof:
https://github.com/hendrikmuhs/ccache-action/blob/2a51777f6f64b7b7bea213601acba8f5f4fdbe03/src/restore.ts#L22-L23
---
 .github/workflows/llvm-project-tests.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/llvm-project-tests.yml b/.github/workflows/llvm-project-tests.yml
index 02b1ab75e960ec..fadaea129e5088 100644
--- a/.github/workflows/llvm-project-tests.yml
+++ b/.github/workflows/llvm-project-tests.yml
@@ -87,7 +87,7 @@ jobs:
           # enough cache space for all the tests to run at once and still
           # fit under the 10 GB limit.
           max-size: 500M
-          key: sccache-${{ matrix.os }}
+          key: ${{ matrix.os }}
           variant: sccache
       - name: Build and Test
         uses: llvm/actions/build-test-llvm-project at main



More information about the llvm-commits mailing list