uploading language po file causes extra lines to be appended

Bug #24 reported by Valient Gough
6
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Carlos Perelló Marín

Bug Description

The server has this patch applied already.

Please, reopen if it still gives you a bad output...

Cheers.

Changed in rosetta:
assignee: None → carlos
status: New → Accepted
Revision history for this message
Valient Gough (vgough) wrote :

I think I found a cause for the "#~" comments (in local files). Note that this is only an explaination of what might cause them to be inserted into files by gettext tools, not a reason they are showing up on Rosetta.

One cause seems to be if msgid strings are removed. In my local files I have just removed one translated string. When I ran the gettext tools to update the .po files, I find that the removed string (along with the translation) is now commented out at the bottom of the file, eg:

#~ msgid "MAC comparison failure in block %lli"
#~ msgstr "Échec de comparaison MAC dans le bloc %lli"

However, this does not explain yet why they are appearing in Rosetta, when no translatable have been neither added nor removed..

Revision history for this message
Carlos Perelló Marín (carlos) wrote : Bug detected and fixed

Hi, after a funny debug session I found the problem.

When we do the export, our code was removing the white space from the end of the multiline msgid:

Original:

msgid ""
"Please select a key size in bits. The cipher you have chosen\n"
"supports sizes from %i to %i bits in increments of %i bits.\n"
"For example: "

Our version:

msgid ""
"Please select a key size in bits. The cipher you have chosen\n"
"supports sizes from %i to %i bits in increments of %i bits.\n"
"For example:"

If you look at the last line, the "For example:" is missing a white space at the end. That caused that the process that does the merge with the template thought that it was a different msgid that does not exists inside the template and thus it marked it that way, as obsolete.

The fix will reach Rosetta server soon.

Thank you for the report and for the help to find the bug.

Cheers.

Changed in rosetta:
status: Accepted → Fixed
Revision history for this message
Carlos Perelló Marín (carlos) wrote :

The server has this patch applied already.

Please, reopen if it still gives you a bad output...

Cheers.

tags: added: iso-testing
tags: removed: iso-testing
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.