[oe] [PATCH (updated)] autotools.bbclass: enhanced oe_runconf() to accept quoted arguments

[Chris Larson]

On Sun, May 23, 2010 at 9:22 AM, Enrico Scholz <
enrico.scholz at sigma-chemnitz.de> wrote:

> Chris Larson <clarson at kergoth.com> writes:
>
> >> This patch allows to pass quoted strings in EXTRA_OECONF.  E.g. with
> >> this patch, it is possible to do
> >>
> >> | EXTRA_OECONF = "--with-build-cflags='${BUILD_CFLAGS}'"
> >>
> >> where 'BUILD_CFLAGS' contains multiple, whitespace separated arguments.
> >>
> >
> > I don't think that doing EXTRA_OECONF =
> > "'--with-build-cflags=${BUILD_CFLAGS}'"
> >
> > is that much more difficult, and works today, unless I'm missing
> > something?
>
> It does not work today... E.g. try the recipe
>
> ---- xxx.bb -----
> EXTRA_OECONF = "--with-build-cflags='-O2 -g3 --help'"
>
> inherit autotools
>
> do_unpack() {
>    mkdir -p ${S}
>    echo 'AC_INIT()' > ${S}/configure.ac
> }
> ----
>
>
> Without the patch, there will be called
>
> | $ ./bitbake -b /tmp/xxx.bb -c configure -f -D
> | ...
> | + ..../configure ...  '--with-build-cflags='\''-O2' -g3 '--help'\'''
>
> and with the patch
>
> | + ..../configure ...  '--with-build-cflags=-O2 -g3 --help'
>
>
I fail to see how this is a problem.  Seems to work fine for every other
app, including make.  Is configure's argument passing that stupid, that it
can't split by = and use the right side for the value?
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics