[Pkg-samba-maint] r1174 - in trunk: . po

[Steve Langasek]

On Wed, Oct 11, 2006 at 07:23:36AM +0200, Christian Perrier wrote:
> > <snip many other similar examples>

> > And this is a *lot* of gratuitous reformatting in the diff.  Christian, what
> > can we do to ensure translation formatting is normalized before committing,
> > so that we're spared unreadable diffs?  I thought that debconf-updatepo
> > would take care of this, but it doesn't seem to do so for the present case
> > (though it does affect the previously-committed Hungarian translation).

> debconf-updatepo does "msgcat" which reformats files.

> I usually run a small script of mine which tests the validity of the
> PO files but does not pass them through msgcat (becacuse it would
> involve a temporary file) so I actually commit files as they're
> submitted.

> Unfortunately, the final format depends on the tool used to edit
> files. KBabel passes the file through msgcat while POEdit doesn't and
> even reformats all strings to a very long single line.

> So, if translators change their tool, we have the effect you saw.

yes, or if different translators work on the same file. :/

> What we could do is certainly run "debconf-updatepo" before
> builds. Running msgcat would be good also, but this can only be a
> manual action by the person committing....

The point would be specifically to run it before commits, to make svn diffs
more legible.  Running it only before builds doesn't seem to have much
benefit, normalization of po files is irrelevant once the pos are converted
into debconf templates at build-time.

> Running debcof-updatepo at each commit would not be good as this would
> change ALL PO files.

IME, it doesn't change any of the po files that I wanted it to, and I had to
run msgcat manually to get reasonably-formatted po output. :/  The only
times I've seen debconf-updatepo result in changes to .po files is when
there were updated strings to be merged in.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
vorlon at debian.org                                   http://www.debian.org/