Launchpad itself

.po parser fails if an \r is not escaped inside a string

Bug #88831 reported by Erdal Ronahi
60
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

Messages like:

#: ../src/gtkdialogs.c:948
#, c-format
msgid ""
"You are about to remove the group %s and all its members from your buddy "
"list. Do you want to continue?"
msgstr ""
"Tu koma %s û hemû endaman ji lîsteya xwe ya hevalan derdixe.^M\n"
"Tu dixwazî bidomînî?"

fail with 'string not terminated' error. because the '^M' before \n is just a non escaped '\r'.

See original description
Revision history for this message
Erdal Ronahi (erdalronahi) wrote :
Revision history for this message
Dan Damian (dand) wrote :

Can confirm this for another po: http://librarian.launchpad.net/6874187/ro.po

Revision history for this message
Elian Myftiu (enlightener) wrote :

I'm having troubles with both these:

http://librarian.launchpad.net/6729130/sq.po

http://librarian.launchpad.net/6904072/help-sq.po

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

Elian, your files work, I guess the problem is with an error we already fixed.

About the others, the error is still there.

Carlos Perelló Marín (carlos)
description: updated
Changed in rosetta:
importance: Undecided → High
status: Unconfirmed → Confirmed
Revision history for this message
Carlos Perelló Marín (carlos) wrote :

Btw, I forgot to say that, even if gettext accepts that file as valid 99,99% of the times, the problem that raises a bug in our parser is actually an error in the translation that would introduce unexpected new lines to translation strings. If you fix it and reimport it, your file should be accepted.

Anyway, we are going to fix our parser because there are some circumstances it's valid and we should allow it.

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

When fixing this, also let's also give a moment's thought to bug 61112.

Revision history for this message
Данило Шеган (danilo) wrote :

Seems this is a duplicate of bug 61096.

Данило Шеган (danilo)
tags: added: rosetta-imports
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Related questions

Remote bug watches

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