[oe] [PATCH 1/5] binconfig.bbclass: search configs in D instead of S

[Paul Menzel]

Am Dienstag, den 19.04.2011, 13:10 +0200 schrieb Andreas Oberritter:
> On 04/17/2011 01:23 AM, Stanislav Brabec wrote:
> > Searching for binconfig in S makes more problems than it
> > brings benefits. Searching in the installed root seems to be more logical.
> > 
> > Fixes following subtle breakages:
> > - binconfig are populated with a different name that upstream intended
> > - populated binconfig files that were not intended to be populated
> > 
> > It fixes at least ncurses, which populates ncurses-config instead of
> > ncurses5-config and ncursesw5-config, causing secondary breakages.

Is `alsa-utils` – as written in your introduction 0/5 – the only
secondary breakage you found? Is that a run time problem? I can build
`alsa-utils-1.0.24.2-r1` just fine using `minimal`.

> > Signed-off-by: Stanislav Brabec <utx at penguin.cz>
> 
> Acked-by: Andreas Oberritter <obi at opendreambox.org>

Acked-by: Paul Menzel <paulepanter at users.sourceforge.net>

I am adding Enrico to the CC since he did a lot of Ncurses work and
maybe can also share his opinion.

> > diff --git a/classes/binconfig.bbclass b/classes/binconfig.bbclass
> > index b3b2236..3d2bf0f 100644
> > --- a/classes/binconfig.bbclass
> > +++ b/classes/binconfig.bbclass
> > @@ -49,7 +49,7 @@ binconfig_package_preprocess () {
> >  SYSROOT_PREPROCESS_FUNCS += "binconfig_sysroot_preprocess"
> >  
> >  binconfig_sysroot_preprocess () {
> > -	for config in `find ${S} -name '${BINCONFIG_GLOB}'`; do
> > +	for config in `find ${D} -name '${BINCONFIG_GLOB}'`; do
> >  		configname=`basename $config`
> >  		install -d ${SYSROOT_DESTDIR}${STAGING_BINDIR_CROSS}
> >  		cat $config | sed ${@get_binconfig_mangle(d)} > ${SYSROOT_DESTDIR}${STAGING_BINDIR_CROSS}/$configname
> >


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110420/209e50c9/attachment-0002.sig>