[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#830308: d-i.debian.org: l10n-sync script breaks headers in individual package files in some situations



Quoting Christian Perrier (bubulle@debian.org):
> Package: d-i.debian.org
> Severity: normal
> Tags: l10n
> 
> It turns out that, as things are right now, the l10n-sync scripts
> appears to break PO files headers in Danish and Belarusian
> translations of many packages, if not all of them.


OK, I have a few more clues about this:

If I revert packages/po/sublevel2/da.po to r70261:

bubulle@kheops:~/src/debian/debian-installer/trunk.test/packages/po/sublevel2 $ svn diff -r HEAD:70261 da.po | patch 
patching file da.po
bubulle@kheops:~/src/debian/debian-installer/trunk.test/packages/po/sublevel2 $ LC_ALL=C msgfmt -c -v -o /dev/null --statistics da.po
da.po:5047:78: syntax error
msgfmt: found 1 fatal error

The offending part:
#. Type: boolean
#. Description
#. :sl2:
#: ../rescue-mode.templates:21001
#| "It is normally a good idea to mount it as it will allow operations such "
#| "as reinstalling the boot loader. However, if the file system on ${FILESYSTEM} is "
#| "corrupt then you may want to avoid mounting it."
msgid ""
"It is normally a good idea to mount it as it will allow operations such as "
"reinstalling the boot loader. However, if the file system on ${FILESYSTEM} "
"is corrupt then you may want to avoid mounting it."
msgstr ""
"Det er normalt en god ide at montere den, da det vil tillade handlinger "
"såsom geninstallation af opstartsindlæseren. Hvis filsystemet på ${FILESYSTEM} er "
"korrept så vil du dog nok ønske at undgå montering af denne."

It should be

#. Type: boolean
#. Description
#. :sl2:
#: ../rescue-mode.templates:21001
#| msgid "It is normally a good idea to mount it as it will allow operations such "
#| "as reinstalling the boot loader. However, if the file system on ${FILESYSTEM} is "
#| "corrupt then you may want to avoid mounting it."
msgid ""
"It is normally a good idea to mount it as it will allow operations such as "
"reinstalling the boot loader. However, if the file system on ${FILESYSTEM} "
"is corrupt then you may want to avoid mounting it."
msgstr ""
"Det er normalt en god ide at montere den, da det vil tillade handlinger "
"såsom geninstallation af opstartsindlæseren. Hvis filsystemet på ${FILESYSTEM} er "
"korrept så vil du dog nok ønske at undgå montering af denne."

(notice the line after "../rescue-mode.templates:21001")

In short, Steve's attempt to fix the translation broke the PO file.

And, later on, l10n-sync choke on this.

And thus, the fix is to make l10n-sync more resilient to this and for
instance have it to test the validity of "master" PO files before
working with them.

More later....

Attachment: signature.asc
Description: PGP signature


Reply to: