From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] virtual-packages: the case for multiple providers selected
Date: Tue, 13 May 2014 22:18:33 +0200 [thread overview]
Message-ID: <20140513221833.016e1d8e@free-electrons.com> (raw)
In-Reply-To: <a67a948f-b496-4385-9796-fc2d079d9641@email.android.com>
Dear Thomas De Schampheleire,
(Would be nice to wrap your e-mails!)
On Tue, 13 May 2014 22:12:59 +0200, Thomas De Schampheleire wrote:
> > - going with the choice means that it is no longer possible to add a
> > new provider in BR2_EXTERNAL without changing the Buildroot source
> > tree, one of the main selling-point of BR2_EXTERNAL to begin with,
>
> I guess it's not possible to put an 'include' statement inside the choice? The included file would then just contain the external options.
What would it include? Remember that include statements in kconfig fail
if the target file doesn't exist.
> > - going with the check means that it will still possible to generate
> > such configurations, which means we'd still get autobuild failures
> > for those (unless the autobuilders are tweaked to recognise this,)
> > while it would be a minimal annoyance to the user.
> >
>
> It doesn't look too difficult to me to handle such prebuild checks in a generic way in the autobuilders.
> Suppose that a magic return code is used if a prebuild check fails (also for the kernel headers check for example), then the autobuilders can check for this magic code and then simply ignore the configuration and generate a new one.
Or we could have a specific make target to allow the autobuilders to
check a configuration before starting the build.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2014-05-13 20:18 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-13 18:09 [Buildroot] virtual-packages: the case for multiple providers selected Yann E. MORIN
2014-05-13 20:05 ` Thomas Petazzoni
2014-05-13 20:12 ` Thomas De Schampheleire
2014-05-13 20:18 ` Thomas Petazzoni [this message]
2014-05-14 7:21 ` Thomas De Schampheleire
2014-05-14 7:30 ` Thomas Petazzoni
2014-05-14 7:34 ` Thomas De Schampheleire
2014-05-14 7:36 ` Thomas Petazzoni
2014-05-13 21:47 ` [Buildroot] [PATCH] infra/pkg-virtual: validate only one provider provides an implementation Yann E. MORIN
2014-05-13 22:05 ` Thomas Petazzoni
2014-05-13 22:14 ` Yann E. MORIN
2014-05-14 8:10 ` Arnout Vandecappelle
2014-05-14 17:35 ` Yann E. MORIN
2014-05-14 8:11 ` [Buildroot] virtual-packages: the case for multiple providers selected Arnout Vandecappelle
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=20140513221833.016e1d8e@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox