After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 785742 - Migrated from Intltool to Gettext
Migrated from Intltool to Gettext
Status: RESOLVED FIXED
Product: gnome-bluetooth
Classification: Core
Component: general
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gnome-bluetooth-general-maint@gnome.bugs
gnome-bluetooth-general-maint@gnome.bugs
Depends on: 785737
Blocks:
 
 
Reported: 2017-08-02 16:08 UTC by Iñigo Martínez
Modified: 2017-08-12 20:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Migrated from Intltool to Gettext (2.63 KB, patch)
2017-08-02 16:08 UTC, Iñigo Martínez
reviewed Details | Review

Description Iñigo Martínez 2017-08-02 16:08:29 UTC
Created attachment 356792 [details] [review]
Migrated from Intltool to Gettext

This patch migrates translations from intltool to gettext. It depends on meson build port patch as it uses meson's i18n module.
Comment 1 Bastien Nocera 2017-08-11 09:39:04 UTC
Review of attachment 356792 [details] [review]:

Did you verify that the .pot file could still be extracted, and that the desktop file did get translated?
Looks good otherwise.
Comment 2 Iñigo Martínez 2017-08-12 17:20:44 UTC
Thank you for reviewing this. Yes, I had checked it, but in case I've missed something I reviewed everything thoroughly again. It's working fine and all the strings are extracted.

Although it may not be interesting enough, meson port also extracts the 'Icon' string from the 'sendto/bluetooth-sendto.desktop.in.in', which is one of the default keywords[0].

[0] http://git.savannah.gnu.org/cgit/gettext.git/tree/gettext-tools/src/read-desktop.c#n643
Comment 3 Piotr Drąg 2017-08-12 17:27:08 UTC
(In reply to Iñigo Martínez from comment #2)
> Although it may not be interesting enough, meson port also extracts the
> 'Icon' string from the 'sendto/bluetooth-sendto.desktop.in.in', which is one
> of the default keywords[0].
> 

That’s https://savannah.gnu.org/support/?108887
Comment 4 Iñigo Martínez 2017-08-12 19:22:47 UTC
Oh! I wasn't aware of the discussion behind the 'Icon' key. Thanks for pointing it to me :)
Comment 5 Bastien Nocera 2017-08-12 19:44:36 UTC
(In reply to Iñigo Martínez from comment #2)
> Thank you for reviewing this. Yes, I had checked it, but in case I've missed
> something I reviewed everything thoroughly again. It's working fine and all
> the strings are extracted.

Looks fine to commit then, thanks.
Comment 6 Iñigo Martínez 2017-08-12 20:11:54 UTC
Done. Pushed as e8ad2ad.