Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#748 closed defect (fixed)

[patch] make a bit more careful with errors in gettext tools

Reported by: hugo Owned by: adrian
Component: Tools Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


The current version of has a rather dumb error handling: if you run on a .po file with errors (for example duplicate IDs or syntax errors), it will throw out the error (usually in the msgmerge step) and leave you with an empty .po file - rather nasty if you just added a load of new translations. The attached patch uses os.popen3 instead of os.popen and watches stderr of the gettext utilities for output. if there is some output, it will write that to stdout and just end the run without overwriting the .po file.

Attachments (1)

make-messages.diff (2.0 KB) - added by hugo 10 years ago.
patch for better error handling in

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by hugo

patch for better error handling in

comment:1 Changed 10 years ago by adrian

  • Resolution set to fixed
  • Status changed from new to closed

(In [1120]) Fixed #748 -- Improved error handling in It no longer clobbers files on error. Thanks, Hugo

Note: See TracTickets for help on using tickets.
Back to Top