Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: packagerEdge.py

Issue 29931199: Issue 7024 - Renamed az translation to az-latn in Microsoft Edge builds (Closed)
Patch Set: Properly handle if translation doesn't exist Created Oct. 29, 2018, 10:29 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packagerEdge.py
===================================================================
--- a/packagerEdge.py
+++ b/packagerEdge.py
@@ -125,6 +125,14 @@
if metadata.has_section('import_locales'):
packagerChrome.import_locales(params, files)
+ # For some mysterious reasons manifoldjs fails with a server error
+ # when building the development build and there is any translation
+ # in az/messages.json for "name_devbuild", however, it works fine
+ # if we use the more specific language code az-latn.
+ az_translation = files.pop('_locales/az/messages.json', None)
+ if az_translation is not None:
+ files['_locales/az-latn/messages.json'] = az_translation
+
files['manifest.json'] = packagerChrome.createManifest(params, files)
if devenv:
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld