All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Anders Darander <anders@chargestorm.se>
Subject: Re: [PATCH v5 7/8] task-base: remove modutils reference.
Date: Fri, 01 Jul 2011 23:35:08 +0100	[thread overview]
Message-ID: <1309559708.20015.566.camel@rex> (raw)
In-Reply-To: <6E60DDFC-9F26-4239-A322-685CF0597C8D@dominion.thruhere.net>

On Fri, 2011-07-01 at 15:32 +0200, Koen Kooi wrote:
> Op 1 jul 2011, om 15:08 heeft Anders Darander het volgende geschreven:
> 
> > Also remove the other kernel24 references.
> > Make everything dependent on kernel26 default.
> > Move the RDEPENDS- and RRECOMMENDS_task-base-kernel26 to _task-base.
> > 
> > Signed-off-by: Anders Darander <anders@chargestorm.se>
> > ---
> > meta/recipes-core/tasks/task-base.bb |   49 ++++++++++-----------------------
> > 1 files changed, 15 insertions(+), 34 deletions(-)
> > 
> >  #
> > # linux-hotplug or none
> > #
> > -HOTPLUG ?= "${@base_contains("MACHINE_FEATURES", "kernel24",  "linux-hotplug","",d)} "
> > +HOTPLUG ?= ""
> 
> I think this one can go completely
> 
> 
> > #
> > # pcmciautils for >= 2.6.13-rc1, pcmcia-cs for others
> > #
> > -PCMCIA_MANAGER ?= "${@base_contains('MACHINE_FEATURES', 'kernel26','pcmciautils','pcmcia-cs',d)} "
> > +PCMCIA_MANAGER ?= "pcmciautils"
> 
> This one can get folded into the task directly

Agreed but I didn't feel it warranted not merging the patches so far.
Please send a follow up patch :)

Cheers,

Richard




  reply	other threads:[~2011-07-01 22:39 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-01 13:07 [PATCH v5 0/8] Linux 3.0 build support Anders Darander
2011-07-01 13:07 ` [PATCH v5 1/8] Remove support for building 2.4 kernels Anders Darander
2011-07-01 13:07 ` [PATCH v5 2/8] image¡kernel.bblass module-init-tools: do not use depmod-2.6 Anders Darander
2011-07-01 13:07 ` [PATCH v5 3/8] modules-init-tools(-cross): update to 3.16 Anders Darander
2011-07-01 13:08 ` [PATCH v5 4/8] kernel.bblass: remove get_kernelmajorversion Anders Darander
2011-07-01 13:08 ` [PATCH v5 5/8] modutils-initscripts: move recipe prior to modutils removal Anders Darander
2011-07-01 13:08 ` [PATCH v5 6/8] modutils: remove modutils Anders Darander
2011-07-01 13:08 ` [PATCH v5 7/8] task-base: remove modutils reference Anders Darander
2011-07-01 13:32   ` Koen Kooi
2011-07-01 22:35     ` Richard Purdie [this message]
2011-07-05  6:54       ` Anders Darander
2011-07-05  8:24         ` Richard Purdie
2011-07-05  9:50           ` Anders Darander
2011-07-01 13:08 ` [PATCH v5 8/8] distro_tracking_fields: remove modutils Anders Darander
2011-07-01 22:35 ` [PATCH v5 0/8] Linux 3.0 build support Richard Purdie

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=1309559708.20015.566.camel@rex \
    --to=richard.purdie@linuxfoundation.org \
    --cc=anders@chargestorm.se \
    --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 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.