All of lore.kernel.org
 help / color / mirror / Atom feed
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 16:07:51 +0100	[thread overview]
Message-ID: <3094369.YRi5CK5LTU@helios> (raw)
In-Reply-To: <CAJTo0LZbiBPiVH2=rTi6ikr3DY0-WQU0GY32Wxi8jBQhhUsnQg@mail.gmail.com>

On Monday 01 July 2013 16:02:56 Burton, Ross wrote:
> On 1 July 2013 15:58, Paul Eggleton <paul.eggleton@linux.intel.com> wrote:
> > 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.
> 
> I mean add a recommends to the udev binary package itself, so you'll
> get udev-cache if you build udev (and keep the existing udev-selection
> logic as-is).

Right, that could work yes.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


      reply	other threads:[~2013-07-01 15:07 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
2013-07-01 15:02         ` Burton, Ross
2013-07-01 15:07           ` Paul Eggleton [this message]

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=3094369.YRi5CK5LTU@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.