Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 476068

Summary: mail-client/evolution-3.8 installs locales in the wrong directory
Product: Gentoo Linux Reporter: Evgeny Bobkin <evgen.ibqn>
Component: [OLD] GNOMEAssignee: Gentoo Linux Gnome Desktop Team <gnome>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: https://git.gnome.org/browse/evolution/tree/configure.ac#n212
See Also: https://bugzilla.gnome.org/show_bug.cgi?id=703733
https://bugs.gentoo.org/show_bug.cgi?id=481390
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 463242    

Description Evgeny Bobkin 2013-07-07 10:07:20 UTC
$ equery files evolution
 * Searching for evolution ...
 * Contents of mail-client/evolution-3.8.3:
...
/usr/locale
/usr/locale/de
/usr/locale/de/LC_MESSAGES
/usr/locale/de/LC_MESSAGES/evolution-3.8.mo
/usr/locale/ru
/usr/locale/ru/LC_MESSAGES
/usr/locale/ru/LC_MESSAGES/evolution-3.8.mo
...

The issue is closely related to the new inttool patch used by us and the line
that evolution just overrides the localedir variable (see the attached url)

localedir='$(prefix)/$(DATADIRNAME)/locale'
AC_SUBST(localedir)

so we should remove those two lines from the configure.ac

We should be warned, that a lot of gnome packages use $(DATADIRNAME) to set the localedir, which is used in

bindtextdomain (GETTEXT_PACKAGE, EVOLUTION_LOCALEDIR);

with the flag -DEVOLUTION_LOCALEDIR=\""$(localedir)"\"	


$(DATADIRNAME) is not set on gentoo anymore.

I have fixed baobab and gnome-clocks upstream already, so once somebody calls eautoreconf for them it will break locale path as well.

Reproducible: Always
Comment 1 Pacho Ramos gentoo-dev 2013-07-07 10:17:08 UTC
Could you report this to upstream too -> bugzilla.gnome.org ?

I guess, we need to drop:
localedir='$(prefix)/$(DATADIRNAME)/locale'
AC_SUBST(localedir)

overwriting 

But I don't understand what you mean in:
"with the flag -DEVOLUTION_LOCALEDIR=\""$(localedir)"\"	
$(DATADIRNAME) is not set on gentoo anymore."

If you can explain it a bit more for me... thanks :)
Comment 2 Evgeny Bobkin 2013-07-07 10:47:55 UTC
(In reply to Pacho Ramos from comment #1)
> Could you report this to upstream too -> bugzilla.gnome.org ?
> 
> I guess, we need to drop:
> localedir='$(prefix)/$(DATADIRNAME)/locale'
> AC_SUBST(localedir)
> 
> overwriting 
> 
> But I don't understand what you mean in:
> "with the flag -DEVOLUTION_LOCALEDIR=\""$(localedir)"\"	
> $(DATADIRNAME) is not set on gentoo anymore."
> 
> If you can explain it a bit more for me... thanks :)

I mean, that there are a lot potential traps in gnome packages, which relies on 
$(DATADIRNAME) variable.

many gnome apps sets in the Makefile.am file a variable, say for example MY_APP_LOCALEDIR to bind the locale path with the app using bindtextdomain (GETTEXT_PACKAGE, MY_APP_LOCALEDIR) function. This variable MY_APP_LOCALEDIR is usually configured with $(DATADIRNAME). So even if all locale related files are placed by our ebuilds in the correct directories like for example for gnome-clocks /usr/locale/de/LC_MESSAGES/gnome-clocks-3.8.mo, your app will be still displayed in english, if you launch it with  LANG="de_DE.utf8" strace -e trace=open gnome-clocks
you will see something like  open("/home/ibqn/opt/gnome//locale/de_DE.utf8/LC_MESSAGES/gnome-clocks.mo", O_RDONLY) = -1 ENOENT (No such file or directory)

I have proposed a fix for glome-clocks upstream only

https://git.gnome.org/browse/gnome-clocks/commit/?id=6ff9908dff50de881dda6adca6bc6672753af058

so if you call aclocal or autoreconf in the current gnome-clocks ebuild you  should break the locale path for GNOMELOCALEDIR
Comment 3 Evgeny Bobkin 2013-07-07 10:50:51 UTC
I meant /usr/share/locale/de/LC_MESSAGES/gnome-clocks-3.8.mo ))
Comment 4 Pacho Ramos gentoo-dev 2013-07-07 11:19:41 UTC
I have just reported:
https://bugzilla.gnome.org/show_bug.cgi?id=703733

if you want to add any note there, feel free ;)
Comment 5 Evgeny Bobkin 2013-07-07 11:29:59 UTC
(In reply to Pacho Ramos from comment #4)
> I have just reported:
> https://bugzilla.gnome.org/show_bug.cgi?id=703733
> 
> if you want to add any note there, feel free ;)

lol, I have also reported this, because you asked me)

https://bugzilla.gnome.org/show_bug.cgi?id=703732
Comment 6 Evgeny Bobkin 2013-07-07 11:39:21 UTC
(In reply to Evgeny Bobkin from comment #5)
> (In reply to Pacho Ramos from comment #4)
> > I have just reported:
> > https://bugzilla.gnome.org/show_bug.cgi?id=703733
> > 
> > if you want to add any note there, feel free ;)
> 
> lol, I have also reported this, because you asked me)
> 
> https://bugzilla.gnome.org/show_bug.cgi?id=703732

I added a path there, which can be used for evolution ebuild, because it already calls eautoreconf.
Comment 7 Pacho Ramos gentoo-dev 2013-07-23 20:56:58 UTC
Solved in 3.8.4