Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#748 closed defect (fixed)

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

Reported by: hugo Owned by: Adrian Holovaty
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: no UI/UX: no


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 16 years ago.
patch for better error handling in

Download all attachments as: .zip

Change History (2)

Changed 16 years ago by hugo

Attachment: make-messages.diff added

patch for better error handling in

comment:1 Changed 16 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(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