[OE-core] [PATCH 1/3] busybox: enable mdev by default

[Khem Raj]

On Thursday, June 02, 2011 09:37:41 AM Richard Purdie wrote:
> On Wed, 2011-06-01 at 20:40 +0000, Otavio Salvador wrote:
> > On Wed, Jun 1, 2011 at 20:37, Phil Blundell <pb at pbcl.net> wrote:
> > > On Wed, 2011-06-01 at 20:09 +0000, Otavio Salvador wrote:
> > >> -# CONFIG_MDEV is not set
> > >> +CONFIG_MDEV=y
> > > 
> > > Per previous discussion, I am still uneasy about this change.  I think
> > > we really need some sort of coherent policy for what exactly the
> > > default busybox configuration in oe-core is meant to be doing, and
> > > then (if necessary) a set of patches to make it match the policy. 
> > > Just flipping random features on and off does not seem like a good way
> > > to proceed.
> > 
> > OE-core has support to mdev as device handling mechanism as such this
> > ought to be enabled by default IMO.
> > 
> > Personally it doesn't matter since I have already overriden it in my
> > internal layer.
> 
> I'm afraid I'm with Phil on this. I don't like the idea of enabling
> something we don't actually use. This really needs to become some kind
> of configure option which would at the same time disable/replace udev so
> the patch in its currently form isn't acceptable.
> 

mdev or udev are image features so probably should be controlled by 
IMAGE_FEATURES or some such
> Cheers,
> 
> Richard
> 
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core