[poky] [PATCH 2/3] kernel.bbclass: introduce UIMAGE_BASE_NAME

Tian, Kevin kevin.tian at intel.com
Mon Dec 20 16:35:51 PST 2010


>From: Richard Purdie [mailto:richard.purdie at linuxfoundation.org]
>Sent: Tuesday, December 21, 2010 5:16 AM
>
>On Sun, 2010-12-19 at 19:21 -0800, Khem Raj wrote:
>> On (17/12/10 08:51), Kevin Tian wrote:
>> >  addtask sizecheck before do_install after do_compile
>> >
>> >  KERNEL_IMAGE_BASE_NAME ?=
>"${KERNEL_IMAGETYPE}-${PV}-${PR}-${MACHINE}-${DATETIME}"
>> > +UIMAGE_BASE_NAME ?= "uImage-${PV}-${PR}-${MACHINE}-${DATETIME}"
>> >  # Don't include the DATETIME variable in the sstate package signatures
>> >  KERNEL_IMAGE_BASE_NAME[vardepsexclude] = "DATETIME"
>> >  KERNEL_IMAGE_SYMLINK_NAME ?= "${KERNEL_IMAGETYPE}-${MACHINE}"
>> > @@ -484,16 +485,16 @@ kernel_do_deploy() {
>> >
>> >  	if test "x${KERNEL_IMAGETYPE}" = "xuImage" ; then
>> >  		if test -e arch/${ARCH}/boot/uImage ; then
>> > -			cp arch/${ARCH}/boot/uImage
>${DEPLOYDIR}/uImage-${PV}-${PR}-${MACHINE}-${DATETIME}.bin
>> > +			cp arch/${ARCH}/boot/uImage
>${DEPLOYDIR}/${UIMAGE_BASE_NAME}.bin
>>
>> why using KERNEL_IMAGE_BASE_NAME does not work here ?
>
>Agreed, this doesn't make sense. I've pushed:
>
>http://git.pokylinux.org/cgit.cgi/poky/commit/?id=2e94e9a4fed33b1c7c5b0ce1dbc566e24
>344280c
>
>since its clear that KERNEL_IMAGETYPE = "uImage".
>

thanks for catching it. I didn't check it carefully. :-)

Thanks
Kevin


More information about the poky mailing list