[Buildroot] [PATCH 2/2] package/ccache: add wrapper for internal toolchain

[Thomas Petazzoni]

Dear Károly Kasza,

On Sun, 26 Apr 2015 11:45:06 +0200, Károly Kasza wrote:

> That is true, so the wrapper for external toolchains also needs the
> extrafiles/hash solution. I simply missed this.

No problem, but it would indeed be nice to have a solution that covers
both cases.

> > I must say I don't quite like the fact that the solution used here is
> > different between internal and external toolchain, but I haven't
> > thought too much of the problem.
> 
> I thought this would be more efficient, but the above statement kills it.

I certainly don't want to kill anything. This ccache problem has been
around for a long time, and it's definitely great to see someone taking
care of this relatively complicated issue.

> > How would ccache work with external toolchains?
> >
> 
> Well, using mtime is out of question because of the BR built wrapper, so it
> requires the
> same solution: a wrapper for ccache and using extrafiles/hash as for the
> internal.

Indeed. But maybe the existing external toolchain wrapper can be used.
We probably don't want to have a shell wrapper call a C wrapper
calling the compiler.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com