All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@uclibc.org>
To: buildroot@busybox.net
Subject: [Buildroot] [pull request v2] Pull request for branch for-2011.02/fix-ccache-support
Date: Tue, 07 Dec 2010 13:26:10 +0100	[thread overview]
Message-ID: <87d3pdu5ul.fsf@macbook.be.48ers.dk> (raw)
In-Reply-To: <20101207001457.56309418@surf> (Thomas Petazzoni's message of "Tue, 7 Dec 2010 00:14:57 +0100")

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

Hi,

 >> Why here and not in the default ~/.ccache? Is the ~/.ccache
 >> directory content ccache-version dependent?

 Thomas> I don't know how ccache-version dependent are the contents of
 Thomas> the cache directory. In the previous ccache integration, the
 Thomas> cache was inside Buildroot build directory (which I thought was
 Thomas> stupid since you then couldn't share the cache between
 Thomas> different Buildroot builds), but I kept the idea of having a
 Thomas> Buildroot-specific location for the cache.

 Thomas> I don't have strong opinion/arguments on that, so if you say we
 Thomas> should use the default cache location, I'll just do it.

Ok, me neither - Just keep it where it is. It just wasn't clear to me
from the commit message if there was any deeper reason behind it.

 >> I remember we had some problems back when we added --sysroot= to TARGET_CC.
 >> The qt package in particular is stripping the --sysroot argument because
 >> of this.

 Thomas> I just tried Qt, and it built fine. It does not use ccache for the
 Thomas> parts compiled for the host (since we don't tell Qt about $(HOSTCXX)),
 Thomas> but it definitely uses the cache for parts compiled for the target.

Ok, thanks.

 Thomas> Here are the results for a Busybox + Qt build, with a
 Thomas> CodeSourcery glibc ARM external toolchain.

 Thomas> Cold cache
 Thomas> ==========

 Thomas> real    7m41.319s
 Thomas> user    37m53.620s
 Thomas> sys     1m31.660s

 Thomas> Hot cache
 Thomas> =========

 Thomas> real    3m4.738s
 Thomas> user    5m34.480s
 Thomas> sys     0m36.160s

Nice!

 Thomas> I am not strongly advocating the usage of the "ccache /path/to/gcc"
 Thomas> solution compared to the symbolic link solutions, but if the first
 Thomas> solution works, I find it better because: 1) it's nicer and 2) it's
 Thomas> easier to implement with external toolchains.

Ok, fine by me if it works.

Care to send a new pull request with the few issues I pointed out fixed,
then I'll commit it?

-- 
Bye, Peter Korsgaard

      reply	other threads:[~2010-12-07 12:26 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-03 19:16 [Buildroot] [pull request v2] Pull request for branch for-2011.02/fix-ccache-support Thomas Petazzoni
2010-12-03 19:16 ` [Buildroot] [PATCH 1/1] ccache: rework ccache management Thomas Petazzoni
2010-12-05 22:47   ` Peter Korsgaard
2010-12-05 22:40 ` [Buildroot] [pull request v2] Pull request for branch for-2011.02/fix-ccache-support Peter Korsgaard
2010-12-06  9:44   ` Bjørn Forsman
2010-12-06 15:08     ` Peter Korsgaard
2010-12-06 19:24       ` Thomas Petazzoni
2010-12-06 19:22     ` Thomas Petazzoni
2010-12-07  0:04       ` Bjørn Forsman
2010-12-07 19:53         ` Thomas Petazzoni
2010-12-08 10:24           ` Bjørn Forsman
2010-12-06 23:14   ` Thomas Petazzoni
2010-12-07 12:26     ` Peter Korsgaard [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87d3pdu5ul.fsf@macbook.be.48ers.dk \
    --to=jacmet@uclibc.org \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.