From: Marco <koansoftware@gmail.com>
To: Darren Hart <dvhart@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [oe][meta-oe][PATCH] kernel.bbclass: kernel_do_install fails with 2.6 kernel recipes
Date: Mon, 07 Jan 2013 18:10:10 +0100 [thread overview]
Message-ID: <50EB0172.7010806@gmail.com> (raw)
In-Reply-To: <50EAF39C.20202@linux.intel.com>
Il 07/01/2013 17:11, Darren Hart ha scritto:
> Hi Marco,
>
> On 01/07/2013 05:39 AM, Marco Cavallini wrote:
>> * old 2.6.x kernels don't have $kerneldir/tools directory
>> * so we got this error:
>> * sed: can't read ...SNIP...linux/2.6.30-r0/image/usr/src/kernel/tools/perf/Makefile: No such file or directory
>> * ERROR: Function failed: do_install
>>
>> Signed-off-by: Marco Cavallini <m.cavallini@koansoftware.com>
>> ---
>> meta/classes/kernel.bbclass | 6 ++++--
>> 1 file changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
>> index aee8a77..06b23e0 100644
>> --- a/meta/classes/kernel.bbclass
>> +++ b/meta/classes/kernel.bbclass
>> @@ -204,8 +204,10 @@ kernel_do_install() {
>> rm -f $kerneldir/$entry
>> done
>>
>> - # Fix SLANG_INC for slang.h
>> - sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' $kerneldir/tools/perf/Makefile
>
> Please include a comment here describing why this was needed. Something
> like:
>
> # Linux 2.6.x kernels don't have a tools directory
>
> (Replace .x with the appropriate version of course)
>
> Otherwise, looks good to me. Thanks!
>
>> + if [ -f $kerneldir/tools/perf/Makefile ]; then
>> + # Fix SLANG_INC for slang.h
>> + sed -i 's#-I/usr/include/slang#-I=/usr/include/slang#g' $kerneldir/tools/perf/Makefile
>> + fi
>> }
>>
>> sysroot_stage_all_append() {
>>
>
Hi Darren,
the comment describing why this was needed was in the patch header, is
in these four lines:
* old 2.6.x kernels don't have $kerneldir/tools directory
* so we got this error:
* sed: can't read
...SNIP...linux/2.6.30-r0/image/usr/src/kernel/tools/perf/Makefile: No
such file or directory
* ERROR: Function failed: do_install
I haven't specified the 2.6 version because I thought that all 2.6
version don't have a 'tools' directory.
BTW I patched it because with 2.6.28 and 2.6.30 it failed.
Cordiali Saluti / Kindest Regards / Mit freundlichen Grüßen
--
Marco Cavallini | KOAN sas | Bergamo - Italia
embedded and real-time software engineering
Phone:+39-035-255.235 - Fax:+39-178-22.39.748
http://www.KoanSoftware.com
next prev parent reply other threads:[~2013-01-07 17:25 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-07 13:39 [oe][meta-oe][PATCH] kernel.bbclass: kernel_do_install fails with 2.6 kernel recipes Marco Cavallini
2013-01-07 16:11 ` Darren Hart
2013-01-07 17:10 ` Marco [this message]
2013-01-07 17:19 ` Darren Hart
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=50EB0172.7010806@gmail.com \
--to=koansoftware@gmail.com \
--cc=dvhart@linux.intel.com \
--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