From: Martin Jansa <martin.jansa@gmail.com>
To: Chris Larson <clarson@kergoth.com>
Cc: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils
Date: Sun, 7 Jul 2013 01:08:40 +0200 [thread overview]
Message-ID: <20130706230840.GE3288@jama> (raw)
In-Reply-To: <CABcZANmrjid3veZXwq_jh24qDaXBjhf3MvNV5yY+6eCg+Gr5Aw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1295 bytes --]
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@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@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
next prev parent reply other threads:[~2013-07-06 23:08 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-05 1:28 [PATCH 0/8] Missing deps discovered in per-recipe builds Martin Jansa
2013-07-05 1:28 ` [PATCH 1/8] (lib)telepathy*: add missing dependency on libxslt-native Martin Jansa
2013-07-05 1:28 ` [PATCH 2/8] gst-plugins-bad: inherit gsettings Martin Jansa
2013-07-05 1:28 ` [PATCH 3/8] gst-plugins-base: add missing dependency on glib-2.0-native Martin Jansa
2013-07-05 1:28 ` [PATCH 4/8] taglib: add missing dependency on zlib Martin Jansa
2013-07-05 1:28 ` [PATCH 5/8] quota: inherit pkgconfig Martin Jansa
2013-07-05 1:28 ` [PATCH 6/8] libpam: " Martin Jansa
2013-07-05 1:28 ` [PATCH 7/8] systemtap: " Martin Jansa
2013-07-05 1:29 ` [PATCH 8/8] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Martin Jansa
2013-07-05 22:43 ` Martin Jansa
2013-07-06 8:39 ` [PATCHv2 1/2] " Martin Jansa
2013-07-06 8:39 ` [PATCHv2 2/2] subversion: explicitly disable sasl Martin Jansa
2013-07-08 14:06 ` Jonathan Liu
2013-07-08 15:01 ` Martin Jansa
2013-07-06 22:56 ` [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Chris Larson
2013-07-06 23:08 ` Martin Jansa [this message]
2013-07-06 23:15 ` [RFC][PATCH] SHLIBSSEARCHDIRS Martin Jansa
2013-07-06 23:17 ` Martin Jansa
2013-07-06 23:16 ` [RFC][PATCH] base.bbclass: add binutils to BASEDEPENDS Martin Jansa
2013-07-06 23:20 ` [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Chris Larson
2013-07-06 23:32 ` Martin Jansa
2013-07-08 3:30 ` Khem Raj
2013-07-05 11:17 ` [PATCH 0/8] Missing deps discovered in per-recipe builds Burton, Ross
2013-07-05 15:28 ` Paul Eggleton
2013-07-05 15:37 ` Burton, Ross
2013-07-06 12:29 ` Martin Jansa
2013-07-06 21:24 ` [RFC][PATCH] test-dependencies: add simple script to detect missing or autoenabled dependencies Martin Jansa
2013-07-06 21:39 ` Martin Jansa
2013-07-08 18:00 ` Randy MacLeod
2013-07-08 19:35 ` Martin Jansa
2013-07-10 12:38 ` [PATCHv2] " Martin Jansa
2013-07-10 12:41 ` Martin Jansa
2013-07-10 14:34 ` Burton, Ross
2013-07-10 14:57 ` Martin Jansa
2013-07-10 13:05 ` sstate-cache and making a package "host-dependent" Mike Looijmans
2013-07-10 17:39 ` Paul Eggleton
2013-07-10 17:45 ` Martin Jansa
2013-07-29 13:17 ` Mike Looijmans
2013-07-11 12:38 ` [PATCHv2] test-dependencies: add simple script to detect missing or autoenabled dependencies Martin Jansa
2013-07-11 13:28 ` Chris Larson
2013-07-11 13:49 ` Martin Jansa
2013-07-11 13:56 ` Chris Larson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130706230840.GE3288@jama \
--to=martin.jansa@gmail.com \
--cc=clarson@kergoth.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox