Patchwork eglibc-locale: Fix depends on binutils

login
register
mail settings
Submitter Richard Purdie
Date Jan. 14, 2014, 11:42 a.m.
Message ID <1389699758.14987.42.camel@ted>
Download mbox | patch
Permalink /patch/64719/
State Accepted
Commit a0bbc06099b4d254a1087b4a6be309518f010e1c
Headers show

Comments

Richard Purdie - Jan. 14, 2014, 11:42 a.m.
The dependency here needs to apply for nativesdk as well as target packages
as the autobuilder just tripped over that. We'd never want a native version
so I'm not sure why the target class override was even present. The dependency
also applies to do_package so lets be explicit about that in case sstate
decides to get clever.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

Patch

diff --git a/meta/recipes-core/eglibc/eglibc-locale.inc b/meta/recipes-core/eglibc/eglibc-locale.inc
index 83569da..d2017b4 100644
--- a/meta/recipes-core/eglibc/eglibc-locale.inc
+++ b/meta/recipes-core/eglibc/eglibc-locale.inc
@@ -8,7 +8,7 @@  LOCALEBASEPN = "${MLPREFIX}eglibc"
 # 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
-DEPENDS_class-target = "virtual/${TARGET_PREFIX}binutils"
+do_package[depends] += "virtual/${TARGET_PREFIX}binutils:do_populate_sysroot"
 
 # Binary locales are generated at build time if ENABLE_BINARY_LOCALE_GENERATION
 # is set. The idea is to avoid running localedef on the target (at first boot)