Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Support for ccache #1248

Merged
merged 2 commits into from
Jul 4, 2019
Merged

Support for ccache #1248

merged 2 commits into from
Jul 4, 2019

Conversation

lbonn
Copy link
Contributor

@lbonn lbonn commented Jul 4, 2019

Would be nice to have that as part of CI, it could maybe halve the test times. I just need to figure out why caching seems to be disabled on our shared runners.

In the meantime, it can already be used for development. The test case is:

apt install ccache
[...]
cd build1; cmake .. -DXXXX && ninja
cd build2; cmake .. -DXXXX && ninja  # (with same CMake options), should be MUCH faster

If it's part of the CMake global definitions, it invalidates all source
files' ccache after each version change (ie: commit).

Exposing the version through a function only requires the definition to
be part of a single file, so most translation units can be cached.

Signed-off-by: Laurent Bonnans <[email protected]>
@lbonn lbonn merged commit 3a13ffa into master Jul 4, 2019
@lbonn lbonn deleted the feat/ccache branch July 4, 2019 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants