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 723138 - Add pre-commit check for po files to ensure translation of keywords ends with semicolon
Add pre-commit check for po files to ensure translation of keywords ends with...
Status: RESOLVED FIXED
Product: website
Classification: Infrastructure
Component: www.gnome.org
current
Other Linux
: Normal enhancement
: ---
Assigned To: GNOME Web maintainers
GNOME Web maintainers
Depends on:
Blocks:
 
 
Reported: 2014-01-28 00:59 UTC by André Klapper
Modified: 2014-07-28 20:06 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description André Klapper 2014-01-28 00:59:31 UTC
Problem comes up from time to time:
https://mail.gnome.org/archives/gnome-i18n/2013-January/msg00030.html
https://mail.gnome.org/archives/gnome-i18n/2013-March/msg00206.html

Somewhere around
https://git.gnome.org/browse/sysadmin-bin/tree/git/pre-receive-check-po#n22 we could parse the po content and pass it to something like

grep -A1 '^msgid\s".\+;.\+;"$' | tail -n 1 | grep '^msgstr\s".\+;.\+[^;]"$'

to refuse translations that end without a semicolon.

Feel free to improve the regex; I am not a hacker(TM).
Comment 1 André Klapper 2014-05-18 11:44:02 UTC
Also filed as https://rt.gnome.org/Ticket/Display.html?id=14101 I just realize.
Comment 2 André Klapper 2014-05-18 11:45:39 UTC

*** This bug has been marked as a duplicate of bug 696489 ***
Comment 3 Olav Vitters 2014-07-28 18:53:29 UTC
$ sed -rn '/^#: .*\.desktop/{:start /\nmsgstr/!{N;b start};/msgid [^\n]+;"/{/msgstr [^\n]+;"/!p}}' sl.po
#: ../data/nautilus.desktop.in.in.h:3
msgid "folder;manager;explore;disk;filesystem;"
msgstr "mapa;upravljalnik;datoteke;raziskovalec;datotečni sistem;disk"
Comment 5 André Klapper 2014-07-28 20:06:55 UTC
Thank you! The cheese in the fridge is all yours!