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

[Saul Wold <sgw@linux.intel.com>]

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@chargestorm.se>
> ---
>   meta/recipes-bsp/u-boot/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
>
> diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2013.07.bb b/meta/recipes-bsp/u-boot/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
> @@ -0,0 +1,31 @@
> +DESCRIPTION = "U-boot bootloader fw_printenv/setenv utils"
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=1707d6db1d42237583f50183a5651ecb"
> +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 = "62c175fbb8a0f9a926c88294ea9f7e88eb898f6c"
> +
> +PV = "v2013.07+git${SRCPV}"
> +
> +SRC_URI = "git://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
> ERROR: Function failed: do_compile (log file is located at /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-world/build/build/tmp/work/qemux86-pokymllib32-linux/lib32-u-boot-fw-utils/v2013.07+gitAUTOINC+62c175fbb8-r0/temp/log.do_compile.4727)
> ERROR: Logfile of failure stored in: /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-world/build/build/tmp/work/qemux86-pokymllib32-linux/lib32-u-boot-fw-utils/v2013.07+gitAUTOINC+62c175fbb8-r0/temp/log.do_compile.4727
> Log data follows:
> | DEBUG: Executing shell function do_compile
> | NOTE: make -j 16 HOSTCC=i586-pokymllib32-linux-gcc  -m32 --sysroot=/srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-world/build/build/tmp/sysroots/lib32-qemux86 HOSTSTRIP=true
> | System not configured - see README
> | make: *** [all] Error 1
> | ERROR: oe_runmake failed
> | WARNING: /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-world/build/build/tmp/work/qemux86-pokymllib32-linux/lib32-u-boot-fw-utils/v2013.07+gitAUTOINC+62c175fbb8-r0/temp/run.do_compile.4727:1 exit 1 from
> |   exit 1
> | ERROR: Function failed: do_compile (log file is located at /srv/home/pokybuild/yocto-autobuilder-new/yocto-slave/nightly-world/build/build/tmp/work/qemux86-pokymllib32-linux/lib32-u-boot-fw-utils/v2013.07+gitAUTOINC+62c175fbb8-r0/temp/log.do_compile.4727)
> NOTE: recipe lib32-u-boot-fw-utils-v2013.07+gitAUTOINC+62c175fbb8-r0: task do_compile: Failed

Sau!