Re: [PATCH 1/5] linux.inc: If the kernel has CONFIG_KERNEL_LZO=y, it may well attempt to use the 'lzop' utility to do the lzo compression, but we should not be relying on that utility being installed on the build machine.

[Denys Dmytriyenko <denis@denix.org>]

On Mon, Dec 19, 2011 at 05:15:36PM +0100, Koen Kooi wrote:
> From: Christopher Larson <chris_larson@mentor.com>

First, a minor nagging - can the commit message be a little shorter? :)

linux.inc: If the kernel has CONFIG_KERNEL_LZO=y, it may well attempt to use
the 'lzop' utility to do the lzo compression, but we should not be relying on
that utility being installed on the build machine.


> This currently affects the linux-omap4 build for omap4430-panda.
> 
> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  recipes-kernel/linux/linux.inc |   11 +++++++++++
>  1 files changed, 11 insertions(+), 0 deletions(-)
> 
> diff --git a/recipes-kernel/linux/linux.inc b/recipes-kernel/linux/linux.inc
> index 9650e68..e8a754f 100644
> --- a/recipes-kernel/linux/linux.inc
> +++ b/recipes-kernel/linux/linux.inc
> @@ -264,3 +264,14 @@ pkg_postrm_kernel-devicetree () {
>  	cd /${KERNEL_IMAGEDEST}; update-alternatives --remove devicetree devicetree-${KERNEL_VERSION} || true
>  }
>  
> +# Automatically depend on lzop-native if CONFIG_KERNEL_LZO is enabled
> +python () {
> +    try:
> +        defconfig = bb.fetch2.localpath('file://defconfig', d)

Second, this will only work with single-config kernels. I guess we'll have to
adopt it for multi-kernel.inc later...


> +    except bb.fetch2.FetchError:
> +        pass
> +    else:
> +        if 'CONFIG_KERNEL_LZO=y\n' in open(defconfig).readlines():
> +            depends = d.getVar('DEPENDS', False)
> +            d.setVar('DEPENDS', depends + ' lzop-native')
> +}
> -- 
> 1.7.2.5
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
>