#15634 closed (fixed)
Malformed Plural-Forms for HR locale
Reported by: | bmihelac | Owned by: | nobody |
---|---|---|---|
Component: | Translations | Version: | dev |
Severity: | Keywords: | blocker | |
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
This is introduced in r15823
error is:
File "/usr/lib/python2.6/gettext.py", line 126, in c2py return eval('lambda n: int(%s)' % plural) File "<string>", line 1 lambda n: int(test(n%10==1 and n%100!=11 , 0 , test( n%10>=2 and n%" "10<=4 and (n%100<10 or n%100>=20) , 1 , 2))) ^ SyntaxError: invalid syntax
Change History (5)
comment:1 by , 14 years ago
Description: | modified (diff) |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 14 years ago
Keywords: | blocker added |
---|---|
milestone: | → 1.3 |
comment:4 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Transifex team reports this also has been fixed in transifex.net. Closing.
Note:
See TracTickets
for help on using tickets.
This seems to be a bug in Transifex when it export the plural forms formula to a .po file and the formula is long enough to cause splitting in certain position. The
\"
tokens in the two lines involved are superfluous and produce errors at run-time when this locale is selected.Removing them and re-generating the .mo files solves this particular problem.
I'd like to propose to fix this in that way (i.e overriding the .po contents with an old style process: manual edit of .po files plus compilation to .mo with msgfmt) to avoid delaying the release waiting for a fix for the Transifex team and a re-export from transifex.net.
W can overwrite the
.po
files with content form Transifex once this is fixed there. Fortunately the manual changes touch this plural-field meta information field and not real translations so the risk of breaking something when we merge thing back later are pretty low.Also, the translation team has 100% translation coverage and it wouldn't be nice to ship 1.3 with a broken translation.
I've reported this in the
#transifex
IRC channel and will open a ticket in their Trac instance.