On Tue, Oct 23, 2012 at 10:49 AM, Christophe Fergeau
<cfergeau@xxxxxxxxxx> wrote:
> On Tue, Oct 23, 2012 at 04:54:52AM +0300, Zeeshan Ali (Khattak) wrote:
>> From: "Zeeshan Ali (Khattak)" <zeeshanak@xxxxxxxxx>
>>
>> Note that we are calling bindtextdomain() from Loader's class init.
>> AFAICT, its a common practice in libraries without any main _init()
>> function to make this call in init of the most common/useful class (e.g
>> gdk-pixbuf and gdbus).
>
> glib tries to do it every time a gettext function is called, but it's
> indeed a different situation as there is no clear entry point in glib.
> You are missing
> #ifdef HAVE_BIND_TEXTDOMAIN_CODESET
> bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
> #endif
> though
>
>> diff --git a/autogen.sh b/autogen.sh
>> index b5997e6..3255713 100755
>> --- a/autogen.sh
>> +++ b/autogen.sh
>> @@ -18,3 +18,4 @@ which gnome-autogen.sh || {
>> }
>>
>> ACLOCAL_FLAGS="$ACLOCAL_FLAGS" USE_GNOME2_MACROS=1 . gnome-autogen.sh --enable-gtk-doc "$@"
>> +intltoolize -f
>> diff --git a/configure.ac b/configure.ac
>> index f021fb6..97f295c 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -12,6 +12,14 @@ LIBOSINFO_COMPILE_WARNINGS
>> GNOME_MAINTAINER_MODE_DEFINES
>> AM_MAINTAINER_MODE([enable])
>>
>> +# i18 support
>> +IT_PROG_INTLTOOL(0.40.0)
>> +
>> +GETTEXT_PACKAGE=AC_PACKAGE_NAME
>> +AC_SUBST(GETTEXT_PACKAGE)
>> +AC_DEFINE_UNQUOTED([GETTEXT_PACKAGE],["$GETTEXT_PACKAGE"],[The name of the gettext domain])
>> +AM_GLIB_GNU_GETTEXT
>
> I think AM_GLIB_GNU_GETTEXT is deprecated these days
> http://blogs.gnome.org/jjardon/2010/10/08/use-upstream-gettext-instead-the-glib-one/
>
>> diff --git a/tools/osinfo-db-validate.c b/tools/osinfo-db-validate.c
>> index 65454d7..31bfeea 100644
>> --- a/tools/osinfo-db-validate.c
>> +++ b/tools/osinfo-db-validate.c
>> @@ -26,6 +26,7 @@
>> #include <glib.h>
>> #include <gio/gio.h>
>> #include <libxml/relaxng.h>
>> +#include <locale.h>
>>
>> #define SCHEMA PKGDATADIR "/schemas/libosinfo.rng"
>>
>> @@ -248,6 +249,7 @@ gint main(gint argc, gchar **argv)
>> GError *error = NULL;
>> gint ret = EXIT_FAILURE;
>>
>> + setlocale(LC_ALL, "");
>> g_type_init();
>
> Standalone binaries generally use
> setlocale (LC_ALL, "");
> textdomain (GETTEXT_PACKAGE);
>
> bindtextdomain (GETTEXT_PACKAGE, GLIB_LOCALE_DIR);
>
> #ifdef HAVE_BIND_TEXTDOMAIN_CODESET
> bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
> #endif
AFAIK they only do the last 3 calls if they have their own gettext
domain but in these binaries we just use libosinfo' gettext domain.
--
Regards,
Zeeshan Ali (Khattak)
FSF member#5124
