[OE-core] [PATCH 1/3] autotools: fix multi-word arguments for EXTRA_OECONF

[Koen Kooi]

Op 7 okt 2011, om 23:02 heeft Christopher Larson het volgende  
geschreven:

> 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 at gmail.com>
> ---
> meta/classes/autotools.bbclass |   14 ++++++--------
> 1 files changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/meta/classes/autotools.bbclass b/meta/classes/ 
> autotools.bbclass
> index a4ce851..135be33 100644
> --- a/meta/classes/autotools.bbclass
> +++ b/meta/classes/autotools.bbclass
> @@ -70,14 +70,12 @@ 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"
> -	else
> -		bbfatal "no configure script found"
> -	fi
> +       if [ -x ${S}/configure ] ; then
> +	       bbnote "Running ${S}/configure ${CONFIGUREOPTS} $ 
> {EXTRA_OECONF} $@"
> +	       ${S}/configure ${CONFIGUREOPTS} ${EXTRA_OECONF} "$@" ||  
> bbfatal "oe_runconf failed"
> +       else
> +	       bbfatal "no configure script found"
> +       f

Was the tab->spaces conversion intentional?