From: Phil Blundell <philb@gnu.org>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH] task-base: conditional wifi and bluetooth tasks in PACKAGES
Date: Wed, 09 Feb 2011 15:22:07 +0000 [thread overview]
Message-ID: <1297264927.2161.15.camel@phil-desktop> (raw)
In-Reply-To: <4D52AC3D.4050906@mentor.com>
On Wed, 2011-02-09 at 08:01 -0700, Tom Rini wrote:
> So... wouldn't making the patch be DISTRO_FEATURES rather than
> MACHINE_FEATURES be what people want?
I think that's still not quite right. As far as I can tell there are
three interesting scenarios that need to be distinguished:
1. DISTRO doesn't want bluetooth at all: bluez shouldn't be built,
nothing should depend on it, and bluetooth should be disabled in all
packages where it's an option. If nothing is using bluez then clearly
it isn't going to be wanted in the bootstrap images.
2. DISTRO does want bluetooth as an option in the feeds but it isn't
needed for bootstrapping (on distros where that's a meaningful concept).
Bluez should be built, packages where it's an option should depend on
it, but task-base/task-bootstrap should not make any special effort to
haul bluez into the installation images.
3. DISTRO wants bluetooth and it is required for bootstrapping. In this
case, evidently something in task-base/task-bootstrap needs to RDEPEND
on it in order to make sure it's available at install time.
I think case (1) corresponds to DISTRO_FEATURES not including bluetooth.
Case (3) corresponds to DISTRO_FEATURES having bluetooth and the MACHINE
declaring that it is bluetooth-capable in some way (either by mentioning
it directly in MACHINE_FEATURES or by mentioning a bus like pci or cf
which could accept a bluetooth card).
The difficulty seems to be that, for case (2), there is currently no way
for a DISTRO to say that it isn't interested in bluetooth for bootstrap
purposes even though the hardware might be capable of it. That might be
a legitimate point of view if all the hardware that it runs on also
supports an easier bootstrap method (e.g. wifi).
If you change the patch to just look at DISTRO_FEATURES then it will
essentially be a no-op since the distros which are having this problem
must already be enabling bluetooth in DISTRO_FEATURES.
All that said, though, I think there is a fairly strong case for just
saying that each distro should provide its own task-bootstrap equivalent
(if it cares about bootstrapping). It seems a bit silly to try to have
a single one-size-fits-all recipe with a zillion little switches that
make it behave differently in a multitude of ways.
p.
next prev parent reply other threads:[~2011-02-09 15:23 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-07 22:13 [PATCH] task-base: conditional wifi and bluetooth tasks in PACKAGES Filip Zyzniewski
2011-02-08 17:29 ` Tom Rini
2011-02-08 18:16 ` Mike Westerhof
2011-02-08 18:23 ` Koen Kooi
2011-02-08 20:39 ` Filip Zyzniewski
2011-02-08 21:04 ` Koen Kooi
2011-02-08 21:22 ` Filip Zyzniewski
2011-02-08 21:25 ` Tom Rini
2011-02-08 21:30 ` Tom Rini
2011-02-09 9:17 ` Filip Zyzniewski
2011-02-08 20:59 ` Douglas Royds
2011-02-08 21:08 ` Eric Bénard
2011-02-08 22:16 ` Koen Kooi
2011-02-08 22:18 ` Eric Bénard
2011-02-09 6:05 ` Filip Zyzniewski
2011-02-09 9:39 ` Marcin Juszkiewicz
2011-02-09 12:59 ` Filip Zyzniewski
2011-02-09 13:28 ` Koen Kooi
2011-02-09 13:43 ` Filip Zyzniewski
2011-02-09 15:01 ` Tom Rini
2011-02-09 15:22 ` Phil Blundell [this message]
2011-02-09 15:46 ` Tom Rini
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=1297264927.2161.15.camel@phil-desktop \
--to=philb@gnu.org \
--cc=openembedded-devel@lists.openembedded.org \
/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.