> -----Original Message----- > From: openembedded-core-boun...@lists.openembedded.org > [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of > Khem Raj > Sent: Monday, January 29, 2018 11:07 PM > To: openembedded-core@lists.openembedded.org > Subject: Re: [OE-core] [PATCH] bitbake.conf: Add comm to HOSTTOOLS > > > > On 1/29/18 10:38 PM, Alexander Kanavin wrote: > > On 01/29/2018 07:47 PM, Niko Mauno wrote: > >> This mitigates following issues during u-boot do_compile() step -- > >> otherwise, if comm is not available, they are quietly ignored: > >> > >> .../scripts/check-config.sh: line 33: comm: command not found > >> .../scripts/check-config.sh: line 39: comm: command not found > >> > >> Since 'comm' is provided by coreutils package, adding it to HOSTTOOLS > >> was considered a lower impact fix compared to adding coreutils-native > >> buildtime dependency to u-boot recipe. > > > > I disagree. Expanding HOSTTOOLS just to cater to the needs of a single > > recipe is not optimal, IMO. > > alternative are > build time dep on coreutils-native.
This works, tested it however u-boot now will depend on coreutils-native. > fix uboot script to not use comm > do not invoke check-config.sh at all. > This might not be an option for us :) > none of them are any better. > Agree with Khem here, if it is not in HOSTTOOLS, it will trickle down to coreutils-native till u-boot script gets patched. Thanks, Manju -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
