Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] bootimg: Use STAGING_KERNEL_DIR
@ 2012-07-05 18:08 Darren Hart
  2012-07-05 20:06 ` Tom Zanussi
  2012-07-06  5:24 ` Kamble, Nitin A
  0 siblings, 2 replies; 5+ messages in thread
From: Darren Hart @ 2012-07-05 18:08 UTC (permalink / raw)
  To: openembedded-core; +Cc: Darren Hart, saul.wold

bootimg.bbclass using STAGING_DIR_HOST/kernel instead of
STAGING_KERNEL_DIR, resulting in build failure of live images.

| install: cannot stat `/usr/local/dev/yocto/fishriver-test/build/tmp/sysroots/fishriver/kernel/bzImage': No such file or directory

Replace it with STAGING_KERNEL_DIR.

UNTESTED - PLEASE TEST PRIOR TO PULL.

Signed-off-by: Darren Hart <dvhart@linux.intel.com>
CC: tom.zanussi@intel.com
CC: saul.wold@intel.com
CC: nitin.a.kamble@intel.com
---
 meta/classes/bootimg.bbclass |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/bootimg.bbclass b/meta/classes/bootimg.bbclass
index 370b378..a4c0e8d 100644
--- a/meta/classes/bootimg.bbclass
+++ b/meta/classes/bootimg.bbclass
@@ -63,7 +63,7 @@ populate() {
 	install -d ${DEST}
 
 	# Install bzImage, initrd, and rootfs.img in DEST for all loaders to use.
-	install -m 0644 ${STAGING_DIR_HOST}/kernel/bzImage ${DEST}/vmlinuz
+	install -m 0644 ${STAGING_KERNEL_DIR}/bzImage ${DEST}/vmlinuz
 
 	if [ -n "${INITRD}" ] && [ -s "${INITRD}" ]; then
 		install -m 0644 ${INITRD} ${DEST}/initrd
-- 
1.7.10.4




^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] bootimg: Use STAGING_KERNEL_DIR
  2012-07-05 18:08 [PATCH] bootimg: Use STAGING_KERNEL_DIR Darren Hart
@ 2012-07-05 20:06 ` Tom Zanussi
  2012-07-05 21:29   ` Darren Hart
  2012-07-06  5:24 ` Kamble, Nitin A
  1 sibling, 1 reply; 5+ messages in thread
From: Tom Zanussi @ 2012-07-05 20:06 UTC (permalink / raw)
  To: Darren Hart; +Cc: saul.wold, openembedded-core

On Thu, 2012-07-05 at 11:08 -0700, Darren Hart wrote:
> bootimg.bbclass using STAGING_DIR_HOST/kernel instead of
> STAGING_KERNEL_DIR, resulting in build failure of live images.
> 
> | install: cannot stat `/usr/local/dev/yocto/fishriver-test/build/tmp/sysroots/fishriver/kernel/bzImage': No such file or directory
> 
> Replace it with STAGING_KERNEL_DIR.
> 
> UNTESTED - PLEASE TEST PRIOR TO PULL.

Fixed the build problem here.

Tested-by: Tom Zanussi <tom.zanussi@intel.com>

> 
> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> CC: tom.zanussi@intel.com
> CC: saul.wold@intel.com
> CC: nitin.a.kamble@intel.com
> ---
>  meta/classes/bootimg.bbclass |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/bootimg.bbclass b/meta/classes/bootimg.bbclass
> index 370b378..a4c0e8d 100644
> --- a/meta/classes/bootimg.bbclass
> +++ b/meta/classes/bootimg.bbclass
> @@ -63,7 +63,7 @@ populate() {
>  	install -d ${DEST}
>  
>  	# Install bzImage, initrd, and rootfs.img in DEST for all loaders to use.
> -	install -m 0644 ${STAGING_DIR_HOST}/kernel/bzImage ${DEST}/vmlinuz
> +	install -m 0644 ${STAGING_KERNEL_DIR}/bzImage ${DEST}/vmlinuz
>  
>  	if [ -n "${INITRD}" ] && [ -s "${INITRD}" ]; then
>  		install -m 0644 ${INITRD} ${DEST}/initrd





^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] bootimg: Use STAGING_KERNEL_DIR
  2012-07-05 20:06 ` Tom Zanussi
@ 2012-07-05 21:29   ` Darren Hart
  2012-07-06  8:39     ` Richard Purdie
  0 siblings, 1 reply; 5+ messages in thread
From: Darren Hart @ 2012-07-05 21:29 UTC (permalink / raw)
  To: Tom Zanussi; +Cc: saul.wold, openembedded-core



On 07/05/2012 01:06 PM, Tom Zanussi wrote:
> On Thu, 2012-07-05 at 11:08 -0700, Darren Hart wrote:
>> bootimg.bbclass using STAGING_DIR_HOST/kernel instead of
>> STAGING_KERNEL_DIR, resulting in build failure of live images.
>>
>> | install: cannot stat `/usr/local/dev/yocto/fishriver-test/build/tmp/sysroots/fishriver/kernel/bzImage': No such file or directory
>>
>> Replace it with STAGING_KERNEL_DIR.
>>
>> UNTESTED - PLEASE TEST PRIOR TO PULL.
> 
> Fixed the build problem here.
> 
> Tested-by: Tom Zanussi <tom.zanussi@intel.com>

Great. I'm getting bags packed and kids ready to go. Can someone please
pull this one in and clean up the commit log to remove the UNTESTED line
and add Tom's Tested-by?

Thanks,

Darren

> 
>>
>> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
>> CC: tom.zanussi@intel.com
>> CC: saul.wold@intel.com
>> CC: nitin.a.kamble@intel.com
>> ---
>>  meta/classes/bootimg.bbclass |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/bootimg.bbclass b/meta/classes/bootimg.bbclass
>> index 370b378..a4c0e8d 100644
>> --- a/meta/classes/bootimg.bbclass
>> +++ b/meta/classes/bootimg.bbclass
>> @@ -63,7 +63,7 @@ populate() {
>>  	install -d ${DEST}
>>  
>>  	# Install bzImage, initrd, and rootfs.img in DEST for all loaders to use.
>> -	install -m 0644 ${STAGING_DIR_HOST}/kernel/bzImage ${DEST}/vmlinuz
>> +	install -m 0644 ${STAGING_KERNEL_DIR}/bzImage ${DEST}/vmlinuz
>>  
>>  	if [ -n "${INITRD}" ] && [ -s "${INITRD}" ]; then
>>  		install -m 0644 ${INITRD} ${DEST}/initrd
> 
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel





^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] bootimg: Use STAGING_KERNEL_DIR
  2012-07-05 18:08 [PATCH] bootimg: Use STAGING_KERNEL_DIR Darren Hart
  2012-07-05 20:06 ` Tom Zanussi
@ 2012-07-06  5:24 ` Kamble, Nitin A
  1 sibling, 0 replies; 5+ messages in thread
From: Kamble, Nitin A @ 2012-07-06  5:24 UTC (permalink / raw)
  To: Darren Hart, openembedded-core@lists.openembedded.org; +Cc: Wold, Saul

ACK, This patch fixes the build. Bitbake is able to complete the creation of the bootable images.

Nitin


> -----Original Message-----
> From: Darren Hart [mailto:dvhart@linux.intel.com]
> Sent: Thursday, July 05, 2012 11:39 PM
> To: openembedded-core@lists.openembedded.org
> Cc: Darren Hart; Zanussi, Tom; Wold, Saul; Kamble, Nitin A
> Subject: [PATCH] bootimg: Use STAGING_KERNEL_DIR
> 
> bootimg.bbclass using STAGING_DIR_HOST/kernel instead of
> STAGING_KERNEL_DIR, resulting in build failure of live images.
> 
> | install: cannot stat
> | `/usr/local/dev/yocto/fishriver-test/build/tmp/sysroots/fishriver/kern
> | el/bzImage': No such file or directory
> 
> Replace it with STAGING_KERNEL_DIR.
> 
> UNTESTED - PLEASE TEST PRIOR TO PULL.
> 
> Signed-off-by: Darren Hart <dvhart@linux.intel.com>
> CC: tom.zanussi@intel.com
> CC: saul.wold@intel.com
> CC: nitin.a.kamble@intel.com
> ---
>  meta/classes/bootimg.bbclass |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/bootimg.bbclass b/meta/classes/bootimg.bbclass
> index 370b378..a4c0e8d 100644
> --- a/meta/classes/bootimg.bbclass
> +++ b/meta/classes/bootimg.bbclass
> @@ -63,7 +63,7 @@ populate() {
>  	install -d ${DEST}
> 
>  	# Install bzImage, initrd, and rootfs.img in DEST for all loaders to use.
> -	install -m 0644 ${STAGING_DIR_HOST}/kernel/bzImage
> ${DEST}/vmlinuz
> +	install -m 0644 ${STAGING_KERNEL_DIR}/bzImage ${DEST}/vmlinuz
> 
>  	if [ -n "${INITRD}" ] && [ -s "${INITRD}" ]; then
>  		install -m 0644 ${INITRD} ${DEST}/initrd
> --
> 1.7.10.4




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] bootimg: Use STAGING_KERNEL_DIR
  2012-07-05 21:29   ` Darren Hart
@ 2012-07-06  8:39     ` Richard Purdie
  0 siblings, 0 replies; 5+ messages in thread
From: Richard Purdie @ 2012-07-06  8:39 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer; +Cc: saul.wold

On Thu, 2012-07-05 at 14:29 -0700, Darren Hart wrote:
> 
> On 07/05/2012 01:06 PM, Tom Zanussi wrote:
> > On Thu, 2012-07-05 at 11:08 -0700, Darren Hart wrote:
> >> bootimg.bbclass using STAGING_DIR_HOST/kernel instead of
> >> STAGING_KERNEL_DIR, resulting in build failure of live images.
> >>
> >> | install: cannot stat `/usr/local/dev/yocto/fishriver-test/build/tmp/sysroots/fishriver/kernel/bzImage': No such file or directory
> >>
> >> Replace it with STAGING_KERNEL_DIR.
> >>
> >> UNTESTED - PLEASE TEST PRIOR TO PULL.
> > 
> > Fixed the build problem here.
> > 
> > Tested-by: Tom Zanussi <tom.zanussi@intel.com>
> 
> Great. I'm getting bags packed and kids ready to go. Can someone please
> pull this one in and clean up the commit log to remove the UNTESTED line
> and add Tom's Tested-by?

Done, thanks!

Richard




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-07-06  8:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-05 18:08 [PATCH] bootimg: Use STAGING_KERNEL_DIR Darren Hart
2012-07-05 20:06 ` Tom Zanussi
2012-07-05 21:29   ` Darren Hart
2012-07-06  8:39     ` Richard Purdie
2012-07-06  5:24 ` Kamble, Nitin A

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox