Patchwork [1/2,v2] autotools: fix multi-word arguments for EXTRA_OECONF

login
register
mail settings
Submitter Christopher Larson
Date Oct. 10, 2011, 9:13 p.m.
Message ID <42ab3ed1bea2342af7061a93c911eced0a6d2bbb.1318281075.git.kergoth@gmail.com>
Download mbox | patch
Permalink /patch/12985/
State New, archived
Headers show

Comments

Christopher Larson - Oct. 10, 2011, 9:13 p.m.
This is needed to better support things like the following (with a
multi-word BUILD_CC):

    EXTRA_OECONF += '"ac_cv_prog_CC_FOR_BUILD=${BUILD_CC}"'

Signed-off-by: Christopher Larson <kergoth@gmail.com>
---
 meta/classes/autotools.bbclass |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

Patch

diff --git a/meta/classes/autotools.bbclass b/meta/classes/autotools.bbclass
index a4ce851..937e2ba 100644
--- a/meta/classes/autotools.bbclass
+++ b/meta/classes/autotools.bbclass
@@ -71,10 +71,8 @@  CONFIGUREOPT_DEPTRACK = "--disable-dependency-tracking"
 
 oe_runconf () {
 	if [ -x ${S}/configure ] ; then
-		cfgcmd="${S}/configure \
-		${CONFIGUREOPTS} ${EXTRA_OECONF} $@"
-		bbnote "Running $cfgcmd..."
-		$cfgcmd || bbfatal "oe_runconf failed" 
+		bbnote "Running ${S}/configure ${CONFIGUREOPTS} ${EXTRA_OECONF} $@"
+		${S}/configure ${CONFIGUREOPTS} ${EXTRA_OECONF} "$@" || bbfatal "oe_runconf failed"
 	else
 		bbfatal "no configure script found"
 	fi