From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 4/4] packagegroup-core-boot: used udev-cache for sysvinit
Date: Mon, 01 Jul 2013 15:58:08 +0100 [thread overview]
Message-ID: <6242225.xYeWPdnF8o@helios> (raw)
In-Reply-To: <CAJTo0LbtyiNhLKvfnn7dE-mUckef7c10wdpj2OKjb5UfrAek5Q@mail.gmail.com>
On Monday 01 July 2013 15:49:00 Burton, Ross wrote:
> On 28 June 2013 11:02, Paul Eggleton <paul.eggleton@linux.intel.com> wrote:
> > I appreciate the intention here but I'm not sure if this is really the
> > appropriate place to put this; packagegroup-core-boot is supposed to only
> > directly pull in the essentials required for booting. This will cause
> > problems for those people using alternative device managers (e.g. busybox
> > mdev) as well.
> >
> > I'm not sure where this should go, but adding it to VIRTUAL-
> > RUNTIME_dev_manager (in the distro config?) could work. Anyone else have
> > any better suggestion?
>
> How about making udev RRECOMMEND udev-cache?
It's not that udev-cache might not be available, rather that packagegroup-
core-boot shouldn't have any kind of reference to it. In any case unless udev
is skipped somehow, an RRECOMMENDS on udev-cache will end up building udev
even if you haven't selected it in VIRTUAL-RUNTIME_dev_manager so that won't
help.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
next prev parent reply other threads:[~2013-07-01 14:58 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-27 22:43 [PATCH 0/4] Updated mc patch and enable udev-cache Saul Wold
2013-06-27 22:43 ` [PATCH 1/4] mc: Don't remove libdir and split helpers into packages Saul Wold
2013-06-27 22:43 ` [PATCH 2/4] packagegroups: Update for split mc packages Saul Wold
2013-06-27 22:43 ` [PATCH 3/4] packagegroup-core-basic: remove hardcode udev Saul Wold
2013-06-27 22:43 ` [PATCH 4/4] packagegroup-core-boot: used udev-cache for sysvinit Saul Wold
2013-06-28 10:02 ` Paul Eggleton
2013-07-01 14:49 ` Burton, Ross
2013-07-01 14:58 ` Paul Eggleton [this message]
2013-07-01 15:02 ` Burton, Ross
2013-07-01 15:07 ` Paul Eggleton
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=6242225.xYeWPdnF8o@helios \
--to=paul.eggleton@linux.intel.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=ross.burton@intel.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.