From: Phil Blundell <pb@pbcl.net>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/3] busybox: enable mdev by default
Date: Fri, 03 Jun 2011 10:57:21 +0100 [thread overview]
Message-ID: <1307095041.2529.234.camel@phil-desktop> (raw)
In-Reply-To: <201106021806.13486.raj.khem@gmail.com>
On Thu, 2011-06-02 at 18:06 -0700, Khem Raj wrote:
> mdev or udev are image features so probably should be controlled by
> IMAGE_FEATURES or some such
Well, yeah, but this is the thing with busybox: it is a monolithic
executable, so you need to decide at configure time what features it's
going to include. Although there is a separate busybox-mdev package, it
just contains the auxiliary scripting and the bulk of the mdev code goes
into busybox itself.
This means that making mdev a pure IMAGE_FEATURE isn't practical, since
it would basically require mdev to be always compiled in on the
offchance that some image might want it. That would lead to code bloat
for those images (probably the majority) which don't.
Equally, having a single straightforward DISTRO_FEATURE to control mdev
is not terribly satisfactory, since some distros might want to configure
busybox with mdev enabled but actually use udev (or nothing) for some or
all of their images.
So, it seems that we really need both: a way for distros to configure
whether busybox is built with mdev on or not, and then an IMAGE_FEATURE
to control whether any given image gets mdev, or udev, or nothing. The
former of those could be done as a DISTRO_FEATURE but, since it's only
going to affect busybox, I'm not sure that there is much benefit in
doing that; it seems like the distros could just as easily take care of
that by direct manipulation of the busybox configuration.
p.
next prev parent reply other threads:[~2011-06-03 10:00 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-01 20:09 [PATCH 0/3] Patches pending on O.S. Systems tree Otavio Salvador
2011-06-01 20:09 ` [PATCH 1/3] busybox: enable mdev by default Otavio Salvador
2011-06-01 20:37 ` Phil Blundell
2011-06-01 20:40 ` Otavio Salvador
2011-06-02 16:37 ` Richard Purdie
2011-06-02 16:40 ` Otavio Salvador
2011-06-03 1:06 ` Khem Raj
2011-06-03 6:37 ` Koen Kooi
2011-06-03 8:24 ` Richard Purdie
2011-06-03 8:59 ` Koen Kooi
2011-06-03 9:50 ` Richard Purdie
2011-06-03 10:42 ` Koen Kooi
2011-06-03 9:57 ` Phil Blundell [this message]
2011-06-01 20:09 ` [PATCH 2/3] qmake_base.bbclass: fix lrelease/lupdate binary names Otavio Salvador
2011-06-02 16:51 ` Paul Eggleton
2011-06-02 17:27 ` Richard Purdie
2011-06-01 20:09 ` [PATCH 3/3] gnutls: link against 'dl' library Otavio Salvador
2011-06-01 20:33 ` Phil Blundell
2011-06-01 20:39 ` Otavio Salvador
2011-06-01 20:53 ` Phil Blundell
2011-06-01 21:00 ` Otavio Salvador
2011-06-01 21:43 ` Saul Wold
2011-06-01 21:52 ` Martin Jansa
2011-06-01 22:19 ` Saul Wold
2011-06-02 6:31 ` Martin Jansa
2011-06-02 9:29 ` Koen Kooi
2011-06-02 9:38 ` Richard Purdie
2011-06-02 11:00 ` [PATCH] gnutls: add --with-libdl-prefix and --with-libpthread-prefix Martin Jansa
2011-06-02 13:53 ` Richard Purdie
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=1307095041.2529.234.camel@phil-desktop \
--to=pb@pbcl.net \
--cc=openembedded-core@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox