diff options
author | Aarni Koskela <akx@iki.fi> | 2019-05-27 12:30:01 +0300 |
---|---|---|
committer | Aarni Koskela <akx@iki.fi> | 2019-05-27 13:09:57 +0300 |
commit | ceb074a8f94e4a0d33bdab3ba54803488c0ce682 (patch) | |
tree | 721d5be8cb4976c88dd4ceae77d64a0f78ab0025 /babel | |
parent | 665c2ac12b95267383e726722d861e54f2d04fee (diff) | |
download | babel-ceb074a8f94e4a0d33bdab3ba54803488c0ce682.tar.gz |
pybabel compile: exit with code 1 if errors were encountered
Fixes #627
Diffstat (limited to 'babel')
-rw-r--r-- | babel/messages/frontend.py | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/babel/messages/frontend.py b/babel/messages/frontend.py index f673f90..9e7c68c 100644 --- a/babel/messages/frontend.py +++ b/babel/messages/frontend.py @@ -182,8 +182,13 @@ class compile_catalog(Command): 'or the base directory') def run(self): + n_errors = 0 for domain in self.domain: - self._run_domain(domain) + for catalog, errors in self._run_domain(domain).items(): + n_errors += len(errors) + if n_errors: + self.log.error('%d errors encountered.' % n_errors) + return (1 if n_errors else 0) def _run_domain(self, domain): po_files = [] @@ -219,6 +224,8 @@ class compile_catalog(Command): if not po_files: raise DistutilsOptionError('no message catalogs found') + catalogs_and_errors = {} + for idx, (locale, po_file) in enumerate(po_files): mo_file = mo_files[idx] with open(po_file, 'rb') as infile: @@ -241,7 +248,8 @@ class compile_catalog(Command): self.log.info('catalog %s is marked as fuzzy, skipping', po_file) continue - for message, errors in catalog.check(): + catalogs_and_errors[catalog] = catalog_errors = list(catalog.check()) + for message, errors in catalog_errors: for error in errors: self.log.error( 'error: %s:%d: %s', po_file, message.lineno, error @@ -252,6 +260,8 @@ class compile_catalog(Command): with open(mo_file, 'wb') as outfile: write_mo(outfile, catalog, use_fuzzy=self.use_fuzzy) + return catalogs_and_errors + class extract_messages(Command): """Message extraction command for use in ``setup.py`` scripts. |