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

[Anders Darander]

I just noted that Otavio already sent a fix for this.

Please, review and apply his fix.

Cheers,
Anders


On 18 September 2013 06:51, Anders Darander <anders.darander at gmail.com>wrote:

>
>
>
> 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
>
>



-- 
Anders Darander
EPO guidelines 1978: "If the contribution to the known art resides
solely in a computer program then the subject matter is not
patentable in whatever manner it may be presented in the claims."
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20130918/69c9dd30/attachment-0002.html>