Patchwork Re-execution of tasks - test report and results

login
register
mail settings
Submitter Richard Purdie
Date April 2, 2012, 3:45 a.m.
Message ID <1333338343.647.8.camel@ted>
Download mbox | patch
Permalink /patch/25059/
State New
Headers show

Comments

Richard Purdie - April 2, 2012, 3:45 a.m.
On Sat, 2012-03-31 at 17:55 +0100, Richard Purdie wrote:
> On Sat, 2012-03-31 at 08:59 -0700, Steve Sakoman wrote:
> > On Sat, Mar 31, 2012 at 8:34 AM, Richard Purdie
> > <richard.purdie@linuxfoundation.org> wrote:
> > 
> > > Hmm, yes. We already have a bug open against git but I'd like to
> > > understand shadow-native and sqlite. If you have the failure logs for
> > > those two, I think it would be worth opening bugs for them.
> > 
> > Details for shadow-native and sqlite below:
> > 
> > shadow-native:
> > 
> > | aclocal: unknown warning category `cross'
> > | configure.in:496: warning: macro `AM_GNU_GETTEXT_VERSION' not found in library
> > | configure.in:497: warning: macro `AM_GNU_GETTEXT' not found in library
> > | autoreconf: configure.in: tracing
> > | autoreconf: configure.in: AM_GNU_GETTEXT_VERSION is used, but not
> > AM_GNU_GETTEXT
> > | autoreconf: running: libtoolize --copy --force
> > | libtoolize: putting auxiliary files in `.'.
> > | libtoolize: copying file `./ltmain.sh'
> > | libtoolize: Consider adding `AC_CONFIG_MACRO_DIR([m4])' to configure.in and
> > | libtoolize: rerunning libtoolize, to keep the correct libtool macros in-tree.
> > | libtoolize: Consider adding `-I m4' to ACLOCAL_AMFLAGS in Makefile.am.
> > | aclocal: unknown warning category `cross'
> > | configure.in:496: warning: macro `AM_GNU_GETTEXT_VERSION' not found in library
> > | configure.in:497: warning: macro `AM_GNU_GETTEXT' not found in library
> > | autoreconf: running:
> > /media/data/yocto/tmp/sysroots/x86_64-linux/usr/bin/autoconf
> > --include=/media/data/yocto/tmp/sysroots/x86_64-linux/usr/share/aclocal-1.11
> > --include=/media/data/yocto/tmp/work/x86_64-linux/shadow-native-4.1.4.3-r3/shadow-4.1.4.3/aclocal-copy/
> > --force --warnings=cross
> > | configure.in:496: error: possibly undefined macro: AM_GNU_GETTEXT_VERSION
> > |       If this token and others are legitimate, please use m4_pattern_allow.
> > |       See the Autoconf documentation.
> > | configure.in:497: error: possibly undefined macro: AM_GNU_GETTEXT
> > | autoreconf: /media/data/yocto/tmp/sysroots/x86_64-linux/usr/bin/autoconf
> > failed with exit status: 1
> > | ERROR: autoreconf execution failed.
> > NOTE: package shadow-native-4.1.4.3-r3: task do_configure: Failed
> > ERROR: Task 1316
> > (/home/steve/source/yocto/poky/meta/recipes-extended/shadow/shadow-native_4.1.4.3.bb,
> > do_configure) failed with exit code '1'
> 
> This looks like a missing gettext dependency...

I think its more subtle than that. I think gettext-minimal-native and
gettext-native are trampling over each other's files. This can probably
be reproduced with:

bitbake gettext-minimal-native
bitbake gettext-native
bitbake gettext-native -c clean
bitbake shadow-native

I think the fix should be something like:


although I'm travelling at the moment and have not tested this yet.

Cheers,

Richard
Paul Eggleton - April 2, 2012, 9:02 a.m.
On Monday 02 April 2012 04:45:43 Richard Purdie wrote:
> diff --git a/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> b/meta/recipes-core/gettext/gettext_0.18.1.1.bb index 4144d2b..0ed5dd9
> 100644
> --- a/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> +++ b/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM =
> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> 
>  PR = "r10"
>  DEPENDS = "libxml2-native gettext-native virtual/libiconv ncurses expat"
> -DEPENDS_virtclass-native = "libxml2-native"
> +DEPENDS_virtclass-native = "libxml2-native gettext-minimal-native"
>  PROVIDES = "virtual/libintl virtual/gettext"
>  PROVIDES_virtclass-native = "virtual/gettext-native"
>  CONFLICTS_${PN} = "proxy-libintl"
> @@ -95,4 +95,8 @@ do_install_append() {
>      rm -f ${D}${libdir}/preloadable_libintl.so
>  }
> 
> +do_install_append_virtclass-native () {
> +	rm ${D}${datadir}/aclocal/*
> +}
> +
>  BBCLASSEXTEND = "native nativesdk"
> 
> although I'm travelling at the moment and have not tested this yet.

I've tested this just now and it does fix the problem. The only thing missing 
is a PR bump.

Cheers,
Paul
Richard Purdie - April 2, 2012, 11:59 a.m.
On Mon, 2012-04-02 at 10:02 +0100, Paul Eggleton wrote:
> On Monday 02 April 2012 04:45:43 Richard Purdie wrote:
> > diff --git a/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> > b/meta/recipes-core/gettext/gettext_0.18.1.1.bb index 4144d2b..0ed5dd9
> > 100644
> > --- a/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> > +++ b/meta/recipes-core/gettext/gettext_0.18.1.1.bb
> > @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM =
> > "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> > 
> >  PR = "r10"
> >  DEPENDS = "libxml2-native gettext-native virtual/libiconv ncurses expat"
> > -DEPENDS_virtclass-native = "libxml2-native"
> > +DEPENDS_virtclass-native = "libxml2-native gettext-minimal-native"
> >  PROVIDES = "virtual/libintl virtual/gettext"
> >  PROVIDES_virtclass-native = "virtual/gettext-native"
> >  CONFLICTS_${PN} = "proxy-libintl"
> > @@ -95,4 +95,8 @@ do_install_append() {
> >      rm -f ${D}${libdir}/preloadable_libintl.so
> >  }
> > 
> > +do_install_append_virtclass-native () {
> > +	rm ${D}${datadir}/aclocal/*
> > +}
> > +
> >  BBCLASSEXTEND = "native nativesdk"
> > 
> > although I'm travelling at the moment and have not tested this yet.
> 
> I've tested this just now and it does fix the problem. The only thing missing 
> is a PR bump.

Thanks for testing. I've gone ahead and merged this into master with a
PR bump added for both gettext and gettext-minimal-native to ensure
sysroots are corrected.

Cheers,

Richard

Patch

diff --git a/meta/recipes-core/gettext/gettext_0.18.1.1.bb b/meta/recipes-core/gettext/gettext_0.18.1.1.bb
index 4144d2b..0ed5dd9 100644
--- a/meta/recipes-core/gettext/gettext_0.18.1.1.bb
+++ b/meta/recipes-core/gettext/gettext_0.18.1.1.bb
@@ -7,7 +7,7 @@  LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 
 PR = "r10"
 DEPENDS = "libxml2-native gettext-native virtual/libiconv ncurses expat"
-DEPENDS_virtclass-native = "libxml2-native"
+DEPENDS_virtclass-native = "libxml2-native gettext-minimal-native"
 PROVIDES = "virtual/libintl virtual/gettext"
 PROVIDES_virtclass-native = "virtual/gettext-native"
 CONFLICTS_${PN} = "proxy-libintl"
@@ -95,4 +95,8 @@  do_install_append() {
     rm -f ${D}${libdir}/preloadable_libintl.so
 }
 
+do_install_append_virtclass-native () {
+	rm ${D}${datadir}/aclocal/*
+}
+
 BBCLASSEXTEND = "native nativesdk"