-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Enable ccache / sccache on our Github CI Actions #436
Comments
Looks like we even have the ccache action already hovering around in the workflow from here: https://github.com/python/cpython/pull/29935/files Just need to use it for the C building bits. |
Actually, since the workflow uses the approach that adds the - name: Add ccache to PATH
run: echo "PATH=/usr/lib/ccache:$PATH" >> $GITHUB_ENV we might already be using ccache for all the subsequent steps including |
Yeah, we definitely already are thanks to @tiran. Here's a comparison of the same commit built on python/cpython with a full cache vs my personal fork with no cache: python/cpythonammaraskar/cpythonI think the action items here are adding the ccache action to the docs and mac build to speed them up and maybe looking at swapping out for sccache so it can be used on Windows as well. |
It also looked like |
We've been using ccache in our CI for a while. Should we keep this open for the Windows CI? |
This should significantly reduce build latency, particularly rebuild latency when minor changes are pushed in PR branches.
It can help across builds, but as the main branch point of a given PR branch is not synchronized how much it helps there will be limited. A lot more likely to help on release branches for bugfix backports as C code in those trees doesn't evolve quickly.
https://github.com/hendrikmuhs/ccache-action has instructions. sccache even claims to support Windows (I've never used it).
The text was updated successfully, but these errors were encountered: