All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] Add BR2_CMAKE_USE_NINJA_BACKEND option
Date: Mon, 30 Jan 2017 10:23:16 +0100	[thread overview]
Message-ID: <20170130102316.2a02ed3a@free-electrons.com> (raw)
In-Reply-To: <901720b443e9bb90750a71c8797bbf0d@openmailbox.org>

Hello,

On Thu, 26 Jan 2017 18:27:36 +0100, C?dric Marie wrote:

> Le 2017-01-25 02:27, Thomas Petazzoni a ?crit?:
> > But then that's a bit annoying because we would have to explicitly
> > set this option to "YES" on all packages that support the Ninja backend
> > (most likely the majority).
> > 
> > I think it would make more sense to default the other way around, i.e
> > default to YES, and set it to NO on the few packages that do not
> > properly support the Ninja backend.
> > 
> > Or maybe better: do not introduce a per-package option for the moment,
> > have only the global one, see in practice how many packages work /
> > don't work and decide if we need a per-package option, and what default
> > value it should have.  
> 
> OK, but it seems that Romain has already noticed a problem with 
> openpowerlink, so it is likely that this option will be necessary - and 
> hopefully default to YES (supported).

Indeed.

> So, why did I want to introduce this option?
> 
> Because I'm not using Buildroot in one-shot mode. I'm developping in a 
> Buildroot environment. So I often make foo-rebuild my package, and I'm 
> interested in saving time for each rebuild.

OK, understood. But do you have numbers showing that "make foo-rebuild"
is actually faster with Ninja ?

In the tests done by Romain, it's only saving 1-2 seconds on the
total build of a package. So I would suspect that the savings on a
partial build are even smaller.

Even if your CMake packages are private, can you give us some numbers
that show the time benefits of Ninja?

> I know this is not what Buildroot is designed for. (Yet, there are a few 
> things that seem to be present for dev mode (rsync with OVERRIDE_SRCDIR 
> for example), so this is not "pure one-shot" either.)

Well, if we do have features such as OVERRIDE_SRCDIR, it's precisely to
make Buildroot usable/useful also during the development. So if there
are things we can improve in this area, it's good to have.

Thanks,

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

  reply	other threads:[~2017-01-30  9:23 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-06 22:37 [Buildroot] [PATCH 1/2] Add BR2_CMAKE_USE_NINJA_BACKEND option Cédric Marie
2017-01-06 22:37 ` [Buildroot] [PATCH 2/2] Update documentation of CMake infrastructure Cédric Marie
2017-01-25  3:27   ` Thomas Petazzoni
2017-01-21 22:25 ` [Buildroot] [PATCH 1/2] Add BR2_CMAKE_USE_NINJA_BACKEND option Romain Naour
2017-01-23 13:39   ` Cédric Marie
2017-01-24 21:48     ` Romain Naour
2017-01-25  1:27       ` Thomas Petazzoni
2017-01-26 17:27         ` Cédric Marie
2017-01-30  9:23           ` Thomas Petazzoni [this message]
2017-02-01 17:01             ` Cédric Marie
2017-02-01 20:12               ` Thomas Petazzoni
2017-02-03 10:44                 ` Cédric Marie
2017-01-25  1:37 ` Thomas Petazzoni
2017-07-11 11:56 ` Thomas Petazzoni
2017-07-11 13:25   ` Cédric Marie
2017-07-11 13:35     ` Thomas Petazzoni

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=20170130102316.2a02ed3a@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --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.