[oe] gettext native failure

[Cliff Brake]

On Tue, Feb 23, 2010 at 10:10 PM, Khem Raj <raj.khem at gmail.com> wrote:
>
> I see. gettext requires libxml2 at buildtime therefore adding
> libxml2-native to DEPENDS
> should solve this. libxml2-native should not depend on gettext-native
> if it does then its a bug.
>
> Dependency chain which pulls in gettext-native is
>
> insane.bbclass -> desktop-file-utils-native -> glib-2.0-native -> gettext-native
>
> so my guess is that it has not yet built libxml2-native when it starts
> to build gettext-native and defaults to whatever is on the build
> system due to parallel'ness of bitbake

Thanks.  I had a mental block thinking linxml2-native depended on
gettext, but it was the parallel build that was getting it build
first.  The following patch fixes it:

diff --git a/recipes/gettext/gettext_0.17.bb b/recipes/gettext/gettext_0.17.bb
index 2f7faa7..fbb479a 100644
--- a/recipes/gettext/gettext_0.17.bb
+++ b/recipes/gettext/gettext_0.17.bb
@@ -4,7 +4,7 @@ SECTION = "libs"
 LICENSE = "GPLv3"
 PR = "r5"
 DEPENDS = "gettext-native virtual/libiconv ncurses expat"
-DEPENDS_virtclass-native = ""
+DEPENDS_virtclass-native = "libxml2-native"
 PROVIDES = "virtual/libintl"
 PROVIDES_virtclass-native = ""
 RCONFLICTS_gettext-libintl = "proxy-libintl"

Are there any objections to pushing this change?  Do I need acks for this?

Thanks,
Cliff