From: Phil Blundell <pb@pbcl.net>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/3] busybox: enable mdev by default
Date: Mon, 23 May 2011 14:06:42 +0100 [thread overview]
Message-ID: <1306156002.2525.79.camel@phil-desktop> (raw)
In-Reply-To: <596fd53943dd383f89abb767d7a3b624e6b20b92.1306152724.git.otavio@ossystems.com.br>
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.
> 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.
p.
next prev parent reply other threads:[~2011-05-23 13:09 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-23 12:13 [PATCH 0/3] Pull request with misc fixes from O.S. Systems tree Otavio Salvador
2011-05-23 12:13 ` [PATCH 1/3] send-pull-request: drop sendemail checks Otavio Salvador
2011-05-23 12:13 ` [PATCH 2/3] busybox: enable mdev by default Otavio Salvador
2011-05-23 13:06 ` Phil Blundell [this message]
2011-05-23 13:10 ` Otavio Salvador
2011-05-23 12:13 ` [PATCH 3/3] syslinux: rdepends on mtools Otavio Salvador
2011-05-25 17:16 ` [PATCH 0/3] Pull request with misc fixes from O.S. Systems tree Saul Wold
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=1306156002.2525.79.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