-
Notifications
You must be signed in to change notification settings - Fork 124
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
About ccache and cache storage #1023
Comments
Hi @mwestphal, I'm the one that put these workflows into place. I used Note that we use I decided to use GitHub releases to store ccache/sccache data for several reasons:
So I came up (in several iterations) with the current system that (mis)uses GitHub Releases to store ccache/sccache data at every commit / PR merge in the main repository. Do you think the current We can switch from |
Thanks for your answer.
Great, github cache behave the same
Great github cache behave the same
It works perfectly actions branches, forks and PRs.
Yes
Windows support I hope that answers your questions. I cann explain how it works in F3D ci if needed. Maybe I'm missing something regarding TTK needs but so far I do really thing that github cacches + sccache works wonders. |
BTW I'm thinking of adding a sccache github action that could be used out of the box, I'll let you know If I do that. |
Hi TTK!
I see that your CI is starting to use ccache, that's great!
However, I see that you started using release to store the caches themselves, I dont think this is the easiest nor the most efficient way to it.
Also, ccache is linux specific, while sccache supports all OSes.
Over at F3D we are using sccache and cache github actions to handle our cache with success and great compilation speedup.
It is done like this:
Full CI is here: https://github.com/f3d-app/f3d-superbuild/blob/main/.github/actions/f3d-superbuild/action.yml#L46
I dont have enough bandwidth to take care of this myself but I'd be happy to explain how it works and why I think this would provide better feature than the solution currently being implemented :)
hth
The text was updated successfully, but these errors were encountered: