From: Anders Darander <anders@chargestorm.se>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH v4 7/8] task-base: remove modutils reference.
Date: Fri, 1 Jul 2011 13:52:08 +0200 [thread overview]
Message-ID: <201107011352.08558.anders@chargestorm.se> (raw)
In-Reply-To: <D041C15B-428E-432D-996B-EF7356B38E29@dominion.thruhere.net>
* Koen Kooi Koen Kooi <koen@dominion.thruhere.net> [07/01/11 09:20 AM]:
> Op 1 jul 2011, om 09:06 heeft Anders Darander het volgende geschreven:
> > Also remove the other kernel24 references.
> > Make everything dependent on kernel26 default.
>
> Since there will be only one of each, can we just completely scrap there
> variables and put the items directly in the tasks?
Sure, we could do that.
One question though, how should I treat
RRECOMMENDS_task-base-kernel26 = "\
kernel-module-nls-utf8 \
kernel-module-input \
kernel-module-uinput \
kernel-module-rtc-dev \
kernel-module-rtc-proc \
kernel-module-rtc-sysfs \
kernel-module-rtc-sa1100 \
kernel-module-unix"
from task-base.bb? Should that part just be new, general
RRECOMMENDS = "\
kernel-module-nls-utf8 \
kernel-module-input \
kernel-module-uinput \
kernel-module-rtc-dev \
kernel-module-rtc-proc \
kernel-module-rtc-sysfs \
kernel-module-rtc-sa1100 \
kernel-module-unix"?
I guess that's the right thing, and I'll create a new pull request with that
merged into the last patch.
Regards,
Anders
--
Anders Darander
ChargeStorm AB T
next prev parent reply other threads:[~2011-07-01 11:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-01 7:05 [PATCH v4 0/8] Linux 3.0 build support Anders Darander
2011-07-01 7:06 ` [PATCH v4 1/8] Remove support for building 2.4 kernels Anders Darander
2011-07-01 7:06 ` [PATCH v4 2/8] image¡kernel.bblass module-init-tools: do not use depmod-2.6 Anders Darander
2011-07-01 7:06 ` [PATCH v4 3/8] modules-init-tools(-cross): update to 3.16 Anders Darander
2011-07-01 7:06 ` [PATCH v4 4/8] kernel.bblass: remove get_kernelmajorversion Anders Darander
2011-07-01 7:06 ` [PATCH v4 5/8] modutils-initscripts: move recipe prior to modutils removal Anders Darander
2011-07-01 7:06 ` [PATCH v4 6/8] modutils: remove modutils Anders Darander
2011-07-01 7:06 ` [PATCH v4 7/8] task-base: remove modutils reference Anders Darander
2011-07-01 7:20 ` Koen Kooi
2011-07-01 11:52 ` Anders Darander [this message]
2011-07-01 12:05 ` Koen Kooi
2011-07-01 22:37 ` Richard Purdie
2011-07-01 12:07 ` Anders Darander
2011-07-01 7:06 ` [PATCH v4 8/8] distro_tracking_fields: remove modutils Anders Darander
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=201107011352.08558.anders@chargestorm.se \
--to=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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox