when Rosetta import fails with POInvalidInputError, a descriptive error should be logged

Bug #2896 reported by Christian Reis
4
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Unassigned

Bug Description

14:27:09 WARNING Error importing Kurdish (ku) translation of gaim in Ubuntu Breezy
+Badger package "gaim"

Traceback (most recent call last):
  File "/srv/launchpad.net/production/launchpad/cronscripts/../lib/canonical/launchpad/database/pofile.py", line 579, in doRawImport
    errors = import_po(self, file, self.rawfilepublished)
  File "/srv/launchpad.net/production/launchpad/cronscripts/../lib/canonical/launchpad/components/poimport.py", line 76, in import_po
    parser.write(file.read())
  File "/srv/launchpad.net/production/launchpad/cronscripts/../lib/canonical/launchpad/components/poparser.py", line 521, in write
    self.parse_line(l)
  File "/srv/launchpad.net/production/launchpad/cronscripts/../lib/canonical/launchpad/components/poparser.py", line 603, in parse_line
    self.append()
  File "/srv/launchpad.net/production/launchpad/cronscripts/../lib/canonical/launchpad/components/poparser.py", line 542, in append
    raise POInvalidInputError('Po file: duplicate msgid '
POInvalidInputError: <unprintable instance object>

The traceback is poor, but the error is still quite clear.

Changed in rosetta:
assignee: nobody → carlos
Changed in rosetta:
assignee: carlos → nobody
status: New → Accepted
Revision history for this message
Dafydd Harries (daf) wrote :

I think the true bug here is that the error reporting mechanism doesn't actually say what the duplicate message ID is. We should give proper error messages to our users so that they can fix bad data themselves.

Changed in rosetta:
status: Confirmed → Fix Released
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.