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

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


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-github-workflow

Author: Craig Hesling (linux4life798)

<details>
<summary>Changes</summary>

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

---
Full diff: https://github.com/llvm/llvm-project/pull/76914.diff


1 Files Affected:

- (modified) .github/workflows/llvm-project-tests.yml (+1-1) 


``````````diff
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

``````````

</details>


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


More information about the llvm-commits mailing list