[OE-core] [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils

[Martin Jansa]

On Sat, Jul 06, 2013 at 03:56:59PM -0700, Chris Larson wrote:
> On Sat, Jul 6, 2013 at 1:39 AM, Martin Jansa <martin.jansa at gmail.com> wrote:
> 
> > * eglibc-collateral.inc inhibits all default deps, but do_package needs
> >   objcopy:
> >   ERROR: objcopy failed with exit code 127 (cmd was
> >   'i586-webos-linux-objcopy' --only-keep-debug
> >   'eglibc-locale/2.17-r0/package/usr/lib/gconv/IBM1166.so'
> >   'eglibc-locale/2.17-r0/package/usr/lib/gconv/.debug/IBM1166.so')
> >   ERROR: Function failed: split_and_strip_files
> >
> 
> Wouldn't it be better to add the necessary dep to the do_package[depends]
> in package.bbclass?

In this case yes, but what about allarch recipes, creating
binutils-cross dependency in their do_package task would probably make
their stamps depending on architecture.

I have similar patch adding virtual/${TARGET_PREFIX}binutils in
BASEDEPENDS, that won't fix this eglibc-locale (because it has
INHIBIT_DEFAULT_DEPS), but it's needed in cases where
gcc-cross.do_configure -> binutils-cross.do_populate_sysroot is skipped
when gcc-cross is reused from sstate.

I'll send RFC for that one now (I was trying to debug why it also brings
dependency on do_package_setscene)

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20130707/28eecf68/attachment-0002.sig>