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

[Otavio Salvador]

On Mon, May 23, 2011 at 13:06, Phil Blundell <pb at pbcl.net> wrote:
> On Mon, 2011-05-23 at 12:13 +0000, Otavio Salvador wrote:
>>  CONFIG_LOSETUP=y
>>  # CONFIG_LSPCI is not set
>>  # CONFIG_LSUSB is not set
>> -# CONFIG_MDEV is not set
>> +CONFIG_MDEV=y
>>  # CONFIG_FEATURE_MDEV_CONF is not set
>>  # CONFIG_FEATURE_MDEV_RENAME is not set
>>  # CONFIG_FEATURE_MDEV_RENAME_REGEXP is not set
>
> This sort of thing, as we've seen previously in classic oe, doesn't
> really scale very well since everyone's requirements for busybox are
> subtly different.  Others will doubtless disagree but, personally, I
> think it would be best for you to just maintain a busybox.conf in your
> own distro layer which sets it up the way that you want.

Yes.

>> diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
>> index 341c5c4..de711d2 100644
>> --- a/meta/recipes-core/busybox/busybox.inc
>> +++ b/meta/recipes-core/busybox/busybox.inc
>> @@ -29,7 +29,6 @@ INITSCRIPT_PARAMS_${PN}-mdev = "start 06 S ."
>>  INITSCRIPT_NAME_${PN}-udhcpd = "busybox-udhcpd"
>>  INITSCRIPT_NAME_${PN}-udhcpc = "busybox-udhcpc"
>>  CONFFILES_${PN}-syslog = "${sysconfdir}/syslog.conf.${PN}"
>> -CONFFILES_${PN}-mdev = "${sysconfdir}/mdev.conf"
>
> Agreed, it seems silly to declare the file as a CONFFILE if we aren't
> shipping it in the first place.

The idea is to not require it to be declared and do the right thing by default.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br