Bug 6816 - Pre commit hooks for i18n
Summary: Pre commit hooks for i18n
Status: NEW
Alias: None
Product: Infrastructure
Classification: Unclassified
Component: Others (show other bugs)
Version: unspecified
Hardware: i586 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Sysadmin Team
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-18 10:10 CEST by Oliver Burger
Modified: 2015-02-06 12:14 CET (History)
4 users (show)

See Also:
Source RPM:
CVE:
Status comment:


Attachments

Description Oliver Burger 2012-07-18 10:10:06 CEST
We should have some pre-commit hooks for the translations in svn.

Namely svn should check the syntax of uploaded po files using
"msgfmt -c"
and it should reject the po files, when they do not pass that check.

That would help us a great deal...
Filip Komar 2012-07-18 10:19:47 CEST

CC: (none) => filip.komar

Comment 1 Marek Laane 2012-07-18 11:21:17 CEST
As in https://bugs.mageia.org/show_bug.cgi?id=1095 I recommend to look at how KDE has done it: http://websvn.kde.org/trunk/kde-common/svn/hooks/test-pofiles.pl?revision=1099588&view=markup probably, though http://websvn.kde.org/trunk/kde-common/svn/hooks/ may contain more interesting ...

CC: (none) => bald

Comment 2 Filip Komar 2012-07-19 20:33:15 CEST
Old mail from misc about that:
https://www.mageia.org/pipermail/mageia-i18n/2011-May/001549.html
Comment 3 Filip Komar 2014-11-05 16:47:32 CET
Besides check -c I suggest also option to unify line length of po files too as they are messy right now:
msgmerge -c -w 80
Comment 4 Rémi Verschelde 2015-02-06 12:14:46 CET
Let's ping Colin on that one after Mageia 5 is released, he's currently quite busy fixing release_blockers ;-)

CC: (none) => remi


Note You need to log in before you can comment on or make changes to this bug.