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

2011-06-03 Thread Koen Kooi
Op 3 jun 2011, om 11:50 heeft Richard Purdie het volgende geschreven: > On Fri, 2011-06-03 at 10:59 +0200, Koen Kooi wrote: >> Op 3 jun 2011, om 10:24 heeft Richard Purdie het volgende geschreven: >> >>> On Fri, 2011-06-03 at 08:37 +0200, Koen Kooi wrote: Op 3 jun 2011, om 03:06 heeft Khem

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

2011-06-03 Thread Phil Blundell
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

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

2011-06-03 Thread Richard Purdie
On Fri, 2011-06-03 at 10:59 +0200, Koen Kooi wrote: > Op 3 jun 2011, om 10:24 heeft Richard Purdie het volgende geschreven: > > > On Fri, 2011-06-03 at 08:37 +0200, Koen Kooi wrote: > >> Op 3 jun 2011, om 03:06 heeft Khem Raj het volgende geschreven: > >> > >>> On Thursday, June 02, 2011 09:37:41

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

2011-06-03 Thread Koen Kooi
Op 3 jun 2011, om 10:24 heeft Richard Purdie het volgende geschreven: > On Fri, 2011-06-03 at 08:37 +0200, Koen Kooi wrote: >> Op 3 jun 2011, om 03:06 heeft Khem Raj het volgende geschreven: >> >>> On Thursday, June 02, 2011 09:37:41 AM Richard Purdie wrote: On Wed, 2011-06-01 at 20:40 +000

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

2011-06-03 Thread Richard Purdie
On Fri, 2011-06-03 at 08:37 +0200, Koen Kooi wrote: > Op 3 jun 2011, om 03:06 heeft Khem Raj het volgende geschreven: > > > On Thursday, June 02, 2011 09:37:41 AM Richard Purdie wrote: > >> On Wed, 2011-06-01 at 20:40 +, Otavio Salvador wrote: > >>> On Wed, Jun 1, 2011 at 20:37, Phil Blundell

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

2011-06-02 Thread Koen Kooi
Op 3 jun 2011, om 03:06 heeft Khem Raj het volgende geschreven: > On Thursday, June 02, 2011 09:37:41 AM Richard Purdie wrote: >> On Wed, 2011-06-01 at 20:40 +, Otavio Salvador wrote: >>> On Wed, Jun 1, 2011 at 20:37, Phil Blundell wrote: On Wed, 2011-06-01 at 20:09 +, Otavio Salvad

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

2011-06-02 Thread Khem Raj
On Thursday, June 02, 2011 09:37:41 AM Richard Purdie wrote: > On Wed, 2011-06-01 at 20:40 +, Otavio Salvador wrote: > > On Wed, Jun 1, 2011 at 20:37, Phil Blundell wrote: > > > On Wed, 2011-06-01 at 20:09 +, Otavio Salvador wrote: > > >> -# CONFIG_MDEV is not set > > >> +CONFIG_MDEV=y > >

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

2011-06-02 Thread Otavio Salvador
On Thu, Jun 2, 2011 at 16:37, Richard Purdie wrote: > 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

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

2011-06-02 Thread Richard Purdie
On Wed, 2011-06-01 at 20:40 +, Otavio Salvador wrote: > On Wed, Jun 1, 2011 at 20:37, Phil Blundell wrote: > > On Wed, 2011-06-01 at 20:09 +, Otavio Salvador wrote: > >> -# CONFIG_MDEV is not set > >> +CONFIG_MDEV=y > > > > Per previous discussion, I am still uneasy about this change. I t

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

2011-06-01 Thread Phil Blundell
On Wed, 2011-06-01 at 20:09 +, 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 doi

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

2011-06-01 Thread Otavio Salvador
On Wed, Jun 1, 2011 at 20:37, Phil Blundell wrote: > On Wed, 2011-06-01 at 20:09 +, 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 th

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

2011-06-01 Thread Otavio Salvador
Since we do not require a configuration file by default we don't force it's addition on the package. If the a layer wants to have it enabled it should also set the CONFFILES for busybox-mdev package. Signed-off-by: Otavio Salvador --- meta/recipes-core/busybox/busybox-1.18.4/defconfig |2 +-