Openembedded Core Discussions
 help / color / mirror / Atom feed
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



  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