[OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Robert P. J. Day
inspired by richard purdie's correction of my original patch, i did a quick grep to see if i could find any oddities related, first, to uses of _append; specifically, where the string being appended did *not* start with a blank. there are certainly proper examples of that, say when one is

Re: [OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Phil Blundell
On Wed, 2012-06-20 at 08:14 -0400, Robert P. J. Day wrote: inspired by richard purdie's correction of my original patch, i did a quick grep to see if i could find any oddities related, first, to uses of _append; specifically, where the string being appended did *not* start with a blank. there

Re: [OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Paul Eggleton
On Wednesday 20 June 2012 13:24:13 Phil Blundell wrote: meta/conf/bitbake.conf:DISTRO_FEATURES_append = ${@oe.utils.distro_features_backfill(d)} That one is probably ok. FYI I designed that not to change DISTRO_FEATURES at all if there was nothing to add -

Re: [OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Robert P. J. Day
On Wed, 20 Jun 2012, Phil Blundell wrote: On Wed, 2012-06-20 at 08:14 -0400, Robert P. J. Day wrote: inspired by richard purdie's correction of my original patch, i did a quick grep to see if i could find any oddities related, first, to uses of _append; specifically, where the string being

Re: [OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Khem Raj
On Wed, Jun 20, 2012 at 5:24 AM, Phil Blundell ph...@gnu.org wrote: meta/classes/multilib.bbclass:PACKAGEFUNCS_append = do_package_qa_multilib That one is a bit weird.  It looks broken but, presumably, it must work otherwise someone would have noticed.  I don't know offhand what is making it

Re: [OE-core] oddities related to _append and _prepend in oe-core

2012-06-20 Thread Robert P. J. Day
On Wed, 20 Jun 2012, Khem Raj wrote: On Wed, Jun 20, 2012 at 5:24 AM, Phil Blundell ph...@gnu.org wrote: meta/classes/multilib.bbclass:PACKAGEFUNCS_append = do_package_qa_multilib That one is a bit weird.  It looks broken but, presumably, it must work otherwise someone would have