Skip to content

Conversation

native-api
Copy link
Contributor

Since the toolchain is git clone'd rather than extracted from a tarball, mtimes of its executables are different each time, invalidating CCache's cache if used in a CI environment.
See e.g. https://travis-ci.community/t/bad-cache-hit-rate-ccache/9400 for an example of this problem.

Since the toolchain is `git clone`'d rather than extracted from a tarball, mtimes of its executables are different each time, invalidating CCache's cache if used in a CI environment.
See e.g. https://travis-ci.community/t/bad-cache-hit-rate-ccache/9400 for an example of this problem.
@lurch
Copy link
Contributor

lurch commented Jul 23, 2020

I'm not sure if this is relevant to our kernel building instructions? Pinging @pelwell for comment
a) I don't think this is specific to compiling the Raspberry Pi kernel
b) surely this is a generic CCache problem, and ccache isn't mentioned anywhere else in our documentation?
c) my comments should be taken with a pinch of salt as I've not used ccache myself 😉

@native-api
Copy link
Contributor Author

native-api commented Jul 23, 2020

This is relevant to the kernel building instructions because

  • they instruct to get the toolchain in an unusual way -- git clone instead of installing a package or unpacking a tarball -- which is the root cause for this problem. It's more relevant to them than CCache documentation because it's them rather than CCache that use an uncommon practice.
  • CCache is the standard product to use to save on long compilation times so it's very likely that someone uses it to build the kernel

@pelwell
Copy link
Contributor

pelwell commented Jul 23, 2020

It seems like good advice to me.

@JamesH65
Copy link
Contributor

Probably not a huge number of people actually doing this, but seems a sane change nevertheless.

Copy link
Contributor

@lurch lurch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Joel Rosdahl <joel@rosdahl.net>
@JamesH65 JamesH65 merged commit 91cb36c into raspberrypi:master Aug 12, 2020
@native-api native-api deleted the patch-1 branch August 13, 2020 07:57
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.

5 participants