-
Notifications
You must be signed in to change notification settings - Fork 88
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
travis: use ccache #61
Conversation
(not sure why I can't select @cladmi for this...) |
now 5 min, before it was 4... |
With that amount of cache misses this isn't surprising. Should I hit rebuild? |
Maybe let's wait for @cladmi to return from his vacation before we merge this ;-) |
(or dismiss it) |
bf63cb7
to
52e1f08
Compare
52e1f08
to
1e7b55e
Compare
There is indeed a limitation that https://github.com/RIOT-OS/RIOT/blob/59b4dcffed6b8f32e6fba01e262e941d7413f7ca/makefiles/docker.inc.mk currently does not map 'CCACHE_DIR' directory or exports the 'CCACHE' variable to the docker container. You can see it in the build output that no mapping is done for it: https://travis-ci.com/RIOT-OS/Tutorials/jobs/192912816#L476-L494 Also, for https://travis-ci.com/RIOT-OS/Tutorials/jobs/192912816#L451-L456 So would need additional configuration. |
It does not look to me like that. The variable you are referring to is called |
Yupp, according to the doc the |
Ok thanks for checking. |
Since the new Github actions are using the Docker image (which uses ccache), we can close this one. |
I'm not sure this works with Docker, but let's give it a try ;-)