[OE-core] [PATCH v2 09/10] binutils.inc: add vardep on multiarch DISTRO_FEATURE
Richard Purdie
richard.purdie at linuxfoundation.org
Fri Sep 28 10:32:03 UTC 2012
On Thu, 2012-09-27 at 20:33 -0500, Matthew McClintock wrote:
> binutils will build differently if this feature is enabled, so
> make the do_configure step depend on it
>
> Signed-off-by: Matthew McClintock <msm at freescale.com>
> ---
> Not sure if we should try to fix via configure options
>
> v2: update commit message a bit
>
> meta/recipes-devtools/binutils/binutils.inc | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-devtools/binutils/binutils.inc b/meta/recipes-devtools/binutils/binutils.inc
> index ee748a4..ff64882 100644
> --- a/meta/recipes-devtools/binutils/binutils.inc
> +++ b/meta/recipes-devtools/binutils/binutils.inc
> @@ -80,6 +80,8 @@ export CC_FOR_BUILD = "LD_LIBRARY_PATH= ${BUILD_CC}"
> export CPP_FOR_BUILD = "${BUILD_CPP}"
> export CFLAGS_FOR_BUILD = "${BUILD_CFLAGS}"
>
> +MULTIARCH := "${@bb.utils.contains("DISTRO_FEATURES", "multiarch", "yes", "no", d)}"
> +do_configure[vardeps] += "MULTIARCH"
> do_configure () {
> (cd ${S}; gnu-configize) || die "Failed to run gnu-configize"
> oe_runconf
This doesn't make sense. oe_runconf should depend on EXTRA_OECONF and
that should depend on DISTRO_FEATURES so the dependency should already
exist?
Cheers,
Richard
More information about the Openembedded-core
mailing list