Re: [Qemu-devel] [PATCH v3 4/4] configure: Add workaround for ccache and clang

[John Snow]

On 03/23/2015 11:14 AM, Peter Maydell wrote:
> On 23 March 2015 at 14:52, John Snow <address@hidden> wrote:
> > On 03/23/2015 09:11 AM, Peter Maydell wrote:
> > > This is really working around a bug in either ccache or
> > > in the way Fedora has configured ccache, so I kind of
> > > feel it ought to be dealt with there. However I don't
> > > object too much to our including the workaround in our
> > > configure...
>
> > I feel like it might be an inescapable consequence of using both ccache and
> > clang together on any system, not just Fedora.
>
> Right, but it's not *QEMU* specific, so it would be better

OK, true.

> if either (a) ccache automatically enabled this if it
> noticed it was being run for clang or (b) Fedora automatically
> enabled this in their aliases which make 'clang' automatically
> be "clang run via ccache".
>
> Is there a bug filed against Fedora for this?
>
> -- PMM

Not that I can see on the RH or ccache/LLVM bugzillas at a quick glance. 
I can try to file something for ccache after I look a little more carefully.

In the meantime, until things improve, I think this workaround is sane 
if we want to deal with clang failures less.

I'll clean up the other minor comments and resubmit.

--js