[OE-core] so what's the deal with "..._append +="?

Robert P. J. Day rpjday at crashcourse.ca
Mon Apr 2 12:07:30 UTC 2012


On Mon, 2 Apr 2012, Richard Purdie wrote:

> On Mon, 2012-04-02 at 07:37 -0400, Robert P. J. Day wrote:
> > pretty sure i mentioned this once before but this just looks weird:
> >
> > $ grep -r "_append +=" *
> > meta/recipes-support/gmp/gmp_5.0.4.bb:SRC_URI_append += "file://use-includedir.patch \
> > meta/recipes-core/eglibc/eglibc-package.inc:FILES_${PN}-dev_append += "${bindir}/rpcgen ${libdir}/*_nonshared.a \
> > meta/recipes-core/eglibc/eglibc-package.inc:FILES_${PN}-staticdev_append += "${libdir}/*.a ${base_libdir}/*.a"
> > meta/recipes-graphics/xorg-lib/libxp_1.0.1.bb:CFLAGS_append += " -I ${S}/include/X11/XprintUtil -I ${S}/include/X11/extensions"
> > $
> >
> >   should those be just simple assignments?
>
> The += does add in a whitespace character. I agree it would be less
> confusing if they were just plain _append = " xxx" though.

  dang, i'd forgotten about the whitespace.  but still, it's
definitely non-intuitive to use what appears to be a double-append
operation.  anyway, i'll leave that for people higher up the food
chain to decide if they want to do anything about it.

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================




More information about the Openembedded-core mailing list