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

Unified Diff: build.py

Issue 29561557: Issue 5763 - Target languages supported by Firefox (Closed)
Patch Set: Fixed undefined variable, put URLS in globals, removed redundand flake8 ignores Created Oct. 5, 2017, 8:54 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 | localeTools.py » ('j') | packagerChrome.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build.py
===================================================================
--- a/build.py
+++ b/build.py
@@ -252,7 +252,6 @@
localeConfig = {
'name_format': 'BCP-47',
'file_format': 'gecko-dtd',
- 'target_platforms': {'gecko'},
'default_locale': packager.defaultLocale
}
elif type in {'chrome', 'gecko-webext'}:
@@ -261,7 +260,6 @@
localeConfig = {
'name_format': 'ISO-15897',
'file_format': 'chrome-json',
- 'target_platforms': {'chrome'},
'default_locale': packager.defaultLocale,
}
else:
@@ -271,8 +269,6 @@
localeConfig = {
'name_format': metadata.get('locales', 'name_format'),
'file_format': metadata.get('locales', 'file_format'),
- 'target_platforms': set(metadata.get('locales',
- 'target_platforms').split()),
'default_locale': metadata.get('locales', 'default_locale')
}
@@ -344,7 +340,7 @@
import buildtools.localeTools as localeTools
for locale, localeDir in localeConfig['locales'].iteritems():
- if locale != localeConfig['default_locale']:
+ if locale != localeConfig['default_locale'].replace('_', '-'):
localeTools.uploadTranslations(localeConfig, metadata, localeDir, locale,
basename, key)
« no previous file with comments | « no previous file | localeTools.py » ('j') | packagerChrome.py » ('J')

Powered by Google App Engine
This is Rietveld