Re: [Tracker] Localization support for tracker

[Luca Ferretti <elle uca libero it>]

Il giorno gio, 09/11/2006 alle 20.47 +0100, Laurent Aguerreche ha
scritto:
> Le jeudi 09 novembre 2006 Ã 18:14 +0100, Laurent Aguerreche a Ãcrit :
> > Le jeudi 09 novembre 2006 Ã 10:24 +0100, Luca Ferretti a Ãcrit :
> > > Il giorno gio, 09/11/2006 alle 00.23 +0100, Laurent Aguerreche ha

> And a new patch...
>
> configure.in was using a wrong macro to get TRACKER_LOCALEDIR correctly
> defined. (AM_GLIB_DEFINE_LOCALEDIR does not work on my system)

AM_GLIB_DEFINE_LOCALEDIR is a macro provided by glib-gettext.m4 (GTK+
glib, not GNU libc) from version 2.1.4.

Maybe you are missing some package (libglib-devel) or you are
experiencing this bug [1] with latest autoconf.

About the autoconf bug: maybe we should request a specific autoconf
version, not use a custom defined macro. Jamie?

> src/libtracker/tracker-search was missing an include of locale.h. I also

Well, following *.h in my system, I've: 

  gi10n.h -> /usr/include/libintl.h -> /usr/include/locale.h

What on your system?

> moved "setlocale(LC_ALL, "")" just upon calls
>
>       bindtextdomain (GETTEXT_PACKAGE, TRACKER_LOCALEDIR);
>         bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
>         textdomain (GETTEXT_PACKAGE);
>
> I think it does not change how *textdomain* functions work but it is
> more clear to me.

Yeah, you could be right. LC_ALL includes LC_MESSAGGES, so it's better
call it before.

Let me investigate other modules.

> (and I include again my file fr.po)

Sorry I forgot the `cvs add`. Now is on cvs (with a couple of fixes).


[1] http://lists.debian.org/debian-gtk-gnome/2006/06/msg00001.html
    http://bugzilla.gnome.org/show_bug.cgi?id=343825