Evolution desktop entry does not load translations

Bug #535650 reported by David Planella
28
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu Translations
Fix Released
High
Unassigned
cdbs (Ubuntu)
Fix Released
High
Martin Pitt
Lucid
Fix Released
High
Martin Pitt
evolution (Ubuntu)
Fix Released
Medium
Martin Pitt
Lucid
Fix Released
Medium
Martin Pitt

Bug Description

Binary package hint: gnome-menus

I've just noticed that the Evolution desktop entry in the menu appears untranslated in ca_ES.UTF-8.

The .desktop file includes the correct Gettext domain, and the .mo file shipped as part of the language packs contains the translations, but they don't seem to get loaded. In /usr/share/applications/desktop*cache the entry is in English.

There where some additions to the desktop entry as part of the DX 10_desktop_shortcuts.patch in Evolution, which might be related to this.

I've only noticed it in Evolution, but I could imagine it will affect other menu entries. If so, feel free to update the bug title accordingly.

David Planella (dpm)
Changed in ubuntu-translations:
status: New → Triaged
importance: Undecided → High
Changed in gnome-menus (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
Martin Pitt (pitti)
Changed in gnome-menus (Ubuntu):
status: New → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

Ah, apparently the DX patch breaks the .desktop file format:

[Desktop Entry]
Name=Evolution
[...]

[Compose Shortcut Group]
Name=Compose New Message
Exec=evolution mailto:
OnlyShowIn=Messaging Menu

[Contacts Shortcut Group]
Name=Contacts
Exec=evolution -c contacts
OnlyShowIn=Messaging Menu

X-Ubuntu-Gettext-Domain=evolution-2.28

So the gettext domain is in the wrong section now.

affects: gnome-menus (Ubuntu) → evolution (Ubuntu)
Changed in evolution (Ubuntu):
importance: Undecided → Medium
Changed in desktop-file-utils (Ubuntu):
status: New → Invalid
Revision history for this message
Martin Pitt (pitti) wrote :

It's better to fix this in a generic fashion in langpack.mk, and then rebuild evolution with that fixed cdbs.

affects: desktop-file-utils (Ubuntu) → cdbs (Ubuntu)
Changed in cdbs (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
importance: Undecided → High
status: Invalid → In Progress
Revision history for this message
Martin Pitt (pitti) wrote :

r143

Changed in cdbs (Ubuntu Lucid):
status: In Progress → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

cdbs uploaded to unapproved. I'll upload a no-change evolution build after that goes in, unless we have other pending evo changes.

Martin Pitt (pitti)
Changed in evolution (Ubuntu Lucid):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cdbs - 0.4.62+nmu1ubuntu6

---------------
cdbs (0.4.62+nmu1ubuntu6) lucid; urgency=low

  * 1/rules/langpack.mk.in: Put X-Ubuntu-Gettext-Domain field at the end of
    the "[Desktop Entry]" block instead of at the end of the file, so that it
    still works with desktop file which have other sections (like evolution
    these days). (LP: #535650)
 -- Martin Pitt <email address hidden> Mon, 15 Mar 2010 19:11:50 +0100

Changed in cdbs (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evolution - 2.28.3-0ubuntu5

---------------
evolution (2.28.3-0ubuntu5) lucid; urgency=low

  * Bump cdbs build dependency, to ensure that we are building with a
    langpack.mk which works with multi-section .desktop files. (LP: #535650)
 -- Martin Pitt <email address hidden> Fri, 19 Mar 2010 15:26:12 +0100

Changed in evolution (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Emmet Hikory (persia) wrote :

Fix in CDBS reverted by cdbs 0.4.62+nmu1ubuntu7

Changed in cdbs (Ubuntu Lucid):
status: Fix Released → Triaged
Revision history for this message
Emmet Hikory (persia) wrote :

Evolution changes undone as a side effect of building 2.28.3-0ubuntu6 against CDBS 0.4.62+nmu1ubuntu7

Changed in evolution (Ubuntu Lucid):
status: Fix Released → Triaged
Revision history for this message
Stephan Peijnik (speijnik) wrote :

The changes in 0.4.62+num1ubuntu6 seem to have broken nautilus as well, see LP: #542438.

Revision history for this message
Martin Pitt (pitti) wrote :

Urgh, sorry for the mess. I tested it on two different files.. I'll test it again on gnome-panel and nautilus the next time.

Changed in cdbs (Ubuntu Lucid):
status: Triaged → In Progress
Changed in evolution (Ubuntu Lucid):
status: Triaged → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package cdbs - 0.4.62+nmu1ubuntu8

---------------
cdbs (0.4.62+nmu1ubuntu8) lucid; urgency=low

  * Add test/i18n: New test package for testing i18n. For now, ship a couple
    of .desktop files in different variants (simple, with translations, with
    initial empty line, with multiple sections).
  * Add test/langpack-1.mk: Verify that langpack.mk's translation stripping
    and domain addition works in all four cases above.
  * 1/rules/langpack.mk.in: Reintroduce .desktop section fix from
    0.4.62+nmu1ubuntu6, but ensure that we do not match the [Desktop Entry]
    section itself. Thanks to Luke Symes for the patch!
    (LP: #535650, #542503)
 -- Martin Pitt <email address hidden> Mon, 22 Mar 2010 09:10:27 +0100

Changed in cdbs (Ubuntu Lucid):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package evolution - 2.28.3-0ubuntu7

---------------
evolution (2.28.3-0ubuntu7) lucid; urgency=low

  * Bump cdbs build dependency, to ensure that we are building with a
    langpack.mk which works with multi-section .desktop files. (LP: #535650)
 -- Martin Pitt <email address hidden> Mon, 22 Mar 2010 10:24:05 +0100

Changed in evolution (Ubuntu Lucid):
status: In Progress → Fix Released
Gabor Kelemen (kelemeng)
Changed in ubuntu-translations:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.