[OE-core] [PATCH 1/1] u-boot-fw-utils: reinstate target recipe for fw_printenv

[Anders Darander]

On 18 September 2013 01:47, Saul Wold <sgw at linux.intel.com> wrote:

> On 09/13/2013 12:21 PM, Anders Darander wrote:
>
>> The latest upgrade changed u-boot-fw-utils to u-boot-fw-utils-cross, which
>> removed the on-target fw_printenv (and fw_setenv).
>>
>> Re-create the updated version of the u-boot-fw-utils recipe.
>>
>> U-Boot does try to strip the fw_printenv binary by default now. To avoid
>> this, without patching the Makefile, we add HOSTSTRIP=true to
>> EXTRA_OEMAKE.
>>
>> The new U-Boot do also require us to configure the build for a specific
>> machine, thus we change the package arch.§
>>
>> Signed-off-by: Anders Darander <anders at chargestorm.se>
>> ---
>>   meta/recipes-bsp/u-boot/u-**boot-fw-utils_2013.07.bb<http://u-boot-fw-utils_2013.07.bb>| 31 ++++++++++++++++++++++
>>   1 file changed, 31 insertions(+)
>>   create mode 100644 meta/recipes-bsp/u-boot/u-**boot-fw-utils_2013.07.bb<http://u-boot-fw-utils_2013.07.bb>
>>
>> diff --git a/meta/recipes-bsp/u-boot/u-**boot-fw-utils_2013.07.bb<http://u-boot-fw-utils_2013.07.bb>b/meta/recipes-bsp/u-boot/
>> u-**boot-fw-utils_2013.07.bb <http://u-boot-fw-utils_2013.07.bb>
>> new file mode 100644
>> index 0000000..f97df71
>> --- /dev/null
>> +++ b/meta/recipes-bsp/u-boot/u-**boot-fw-utils_2013.07.bb<http://u-boot-fw-utils_2013.07.bb>
>> @@ -0,0 +1,31 @@
>> +DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
>> +LICENSE = "GPLv2+"
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=**1707d6db1d42237583f50183a5651e*
>> *cb"
>> +SECTION = "bootloader"
>> +DEPENDS = "mtd-utils"
>> +
>> +# This revision corresponds to the tag "v2013.07"
>> +# We use the revision in order to avoid having to fetch it from the
>> +# repo during parse
>> +SRCREV = "**62c175fbb8a0f9a926c88294ea9f7e**88eb898f6c"
>> +
>> +PV = "v2013.07+git${SRCPV}"
>> +
>> +SRC_URI = "git://git.denx.de/u-boot.git;**branch=master;protocol=git<http://git.denx.de/u-boot.git;branch=master;protocol=git>
>> "
>> +
>> +S = "${WORKDIR}/git"
>> +
>> +EXTRA_OEMAKE = 'HOSTCC="${CC}" HOSTSTRIP="true"'
>> +
>> +do_compile () {
>> +  oe_runmake ${UBOOT_MACHINE}
>> +  oe_runmake env
>> +}
>> +
>> +do_install () {
>> +  install -d ${D}${base_sbindir}
>> +  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_**
>> printenv
>> +  install -m 755 ${S}/tools/env/fw_printenv ${D}${base_sbindir}/fw_setenv
>> +}
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>
>>
> I think this might need a COMPATIBLE_MACHINE or some check for correct
> systems
>

Ah, you're correct.

I was looking too much on the old recipe and my working machine... As we're
currently making the package machine specific, we definitely need this
test. v2 on its way...

Cheers,
Anders
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20130918/a7bd4e66/attachment-0002.html>