[ELDK] [PATCH v1 1/2] META-ELDK: Linux kernel build-depends on the bc(1) command

[Marek Vasut]

On Tuesday, April 22, 2014 at 08:02:44 PM, Gerhard Sittig wrote:
> On Tue, 2014-04-22 at 18:46 +0200, Marek Vasut wrote:
> > On Tuesday, April 22, 2014 at 04:14:30 PM, Gerhard Sittig wrote:
> > > diff --git a/meta-eldk/recipes-kernel/linux/linux.inc
> > > b/meta-eldk/recipes-kernel/linux/linux.inc index
> > > 7c08af4a7a69..905849bf20fa 100644
> > > --- a/meta-eldk/recipes-kernel/linux/linux.inc
> > > +++ b/meta-eldk/recipes-kernel/linux/linux.inc
> > > @@ -24,6 +24,7 @@ KERNEL_DIR_SUFFIX = "${@'xenomai' if '${PN}' ==
> > > 'linux-xenomai' else 'eldk'}" KERNEL_SRC_PATH_append =
> > > '-${KERNEL_DIR_SUFFIX}'
> > > 
> > >  KERNEL_LINK_PATH = "/usr/src/kernel"
> > > 
> > > +DEPENDS_append = " bc-native "
> > > 
> > >  DEPENDS_append = " u-boot-mkimage-native "
> > >  DEPENDS_append_em-x270 = " mtd-utils "
> > 
> > Nice find, but
> > 1) this should be fixed in the Yocto recipe and submitted into mainline
> > Yocto 2) you should depend on 'bc' , not 'bc-native' as the -native
> > suffix should be
> > 
> >    added by Yocto, no ?
> 
> Note that the fix applies to the ELDK layer's linux.inc file.
> Yocto has 'DEPENDS += "xz-native bc-native"' in the eldk-5.5
> meta/recipes-kernel/linux/linux-yocto.inc file.
> 
> I ran into the missing bc(1) dependency.  Haven't missed xz(1),
> can't tell which platform or configuration of the Linux kernel
> requires this (LZMA related tool needed at compile time?).

Nice. I think the XZ is needed for XZ compression support in kernel 
decompressor. Add the XZ dependency as well so we're in sync with yocto and add:

Acked-by: Marek Vasut <marex at denx.de>

Best regards,
Marek Vasut