Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use sccache to cache build results #4486

Merged
merged 2 commits into from
Mar 12, 2023

Conversation

sylvestre
Copy link
Contributor

instead of Swatinem/rust-cache@v2

After that PR, GNU's jobs are next

@sylvestre
Copy link
Contributor Author

We look good:

/opt/hostedtoolcache/sccache/0.4.0-pre.9/x64/sccache --show-stats
Compile requests                    377
Compile requests executed           240
Cache hits                          226
Cache hits (C/C++)                   48
Cache hits (Rust)                   178
Cache misses                         13
Cache misses (Rust)                  13
Cache timeouts                        0
Cache read errors                     0
Forced recaches                       0
Cache write errors                    0
Compilation failures                  1
Cache errors                          0
Non-cacheable compilations            0
Non-cacheable calls                 135
Non-compilation calls                 2
Unsupported compiler calls            0
Average cache write               0.143 s
Average compiler                  0.582 s
Average cache read hit            0.000 s
Failed distributed compilations       0

Non-cacheable reasons:
incremental                          75
crate-type                           55
unknown source language               4
-                                     1

Cache location                  ghac, name: sccache-v0.4.0-pre.9, prefix: /sccache/
Version (client)                0.4.0-pre.9

@tertsdiepraam
Copy link
Member

Cool! You're still using Swatinem/rust-cache in a few places, do we need both there? Or is that just to get access to the Rust toolchain?

@sylvestre
Copy link
Contributor Author

It seems that they are caching different things

@tertsdiepraam
Copy link
Member

Alright! There's no harm in having both either I suppose.

@tertsdiepraam tertsdiepraam merged commit 758e4b0 into uutils:main Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants