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

Delta Between Two Patch Sets: mozharness/abb/transform_locales.py

Issue 29327949: Issue 3047 - Change default search engines (Closed)
Left Patch Set: regex enhancement, code duplication reduction. Created Sept. 16, 2015, 2:14 p.m.
Right Patch Set: Fixed even more review issues Created Sept. 22, 2015, 10:53 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Right: Side by side diff | Download
« no previous file with change/comment | « mozharness/abb/__init__.py ('k') | mozharness/mozilla/l10n/multi_locale_build.py » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 # This file is part of Adblock Plus
2 # Copyright (C) 2006-2015 Eyeo GmbH
3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation.
7 #
8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # GNU General Public License for more details.
12 #
13 # You should have received a copy of the GNU General Public License
14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
15
16 import os
17 import re
18
19 _LOCALE_RE = re.compile("^[a-z]{2}(-[A-Z]{2})?$")
20 _SEARCH_PROPS_RE = re.compile("^browser\.search\."
21 "(defaultenginename|order\.).*$")
22 _SHORTNAME_RE = re.compile("^<ShortName>(.*)</ShortName>$")
23
24 _SEARCHPLUGINS_PATH = os.path.join("browser", "searchplugins")
25 _LIST_TXT_PATH = os.path.join(_SEARCHPLUGINS_PATH, "list.txt")
26 _REGION_PROPS_PATH = os.path.join("browser", "region.properties")
27
28 _DEFAULT_LOCALE = "en-US"
29
30 _SEARCH_ENGINE_ORDER = {
31 "en-US": ["duckduckgo",
32 "yahoo",
33 "google",
34 "wikipedia",
35 "amazon"
36 ],
37 "zh-CN": ["baidu",
38 "duckduckgo",
39 "yahoo",
40 "google",
41 "wikipedia",
42 "amazon"
43 ]
44 }
45
46
47 def _get_shortname_from_id(needle, engine_ids, engine_names):
48 """Fuzzy finds needle in engine_ids and returns ShortName"""
49 for engine in engine_ids:
50 if engine.startswith(needle):
51 return engine_names[engine]
52 return None
53
54
55 def _write_lines(filename, lines):
56 """Writes lines into file appending \\n"""
57 with open(filename, "w") as fd:
58 for l in lines:
59 fd.write("%s\n" % l)
60
61
62 def _transform_locale(locale, path, logger):
63 logger.info("Processing locale '%s'..." % locale)
64
65 # Check for list.txt existence
66 list_file_path = os.path.join(path, _LIST_TXT_PATH)
67 if not os.path.exists(list_file_path):
68 logger.fatal("Missing 'list.txt' for locale '%s'" % locale)
69
70 # Check for region.properties existence
71 region_file_path = os.path.join(path, _REGION_PROPS_PATH)
72 if not os.path.exists(region_file_path):
73 logger.fatal("Missing 'region.properties' for locale '%s'" % locale)
74
75 # Get whitelist and build regex
76 whitelist = _SEARCH_ENGINE_ORDER.get(locale,
77 _SEARCH_ENGINE_ORDER[_DEFAULT_LOCALE])
78 white_re = re.compile("^(%s).*$" % "|".join(whitelist))
79
80 # Read engine IDs from list.txt, discard engines not on whitelist
81 engine_ids = []
82 with open(list_file_path, "r") as fd:
83 for line in fd:
84 line = line.strip()
85 if len(line) > 0:
86 if white_re.match(line):
87 engine_ids.append(line)
88 else:
89 logger.info("Removing '%s'" % line)
90
91 # Make sure we still have search engines left
92 if len(engine_ids) == 0:
93 logger.fatal("No search engines left over for '%s'" % locale)
94
95 # 'Parse' XML to get matching 'ShortName' for all engine IDs
96 engine_names = {}
97 for eid in engine_ids:
98 xml_file_path = os.path.join(path, _SEARCHPLUGINS_PATH, "%s.xml" % eid)
99 if os.path.exists(xml_file_path):
100 short_name = None
101 with open(xml_file_path, "r") as fd:
102 for line in fd:
103 line = line.strip()
104 match = _SHORTNAME_RE.match(line)
105 if match:
106 short_name = match.group(1).strip()
107
108 if not short_name:
109 logger.fatal("No ShortName defined for '%s' in '%s" %
110 (eid, locale))
111 engine_names[eid] = short_name
112 else:
113 logger.fatal("XML definiton for '%s' in '%s' missing" %
114 (eid, locale))
115
116 logger.info("Remaining engine IDs: %s" % ", ".join(engine_ids))
117
118 # Create search engine order with real engine names
119 engine_order = []
120 for eid in whitelist:
121 sn = _get_shortname_from_id(eid, engine_ids, engine_names)
122 if sn:
123 engine_order.append(sn)
124
125 logger.info("Resulting ordered list: %s" % (", ".join(engine_order)))
126
127 # Read region.properties and remove browser.search.* lines
128 props = []
129 with open(region_file_path, "r") as fd:
130 for line in fd:
131 line = line.rstrip("\r\n")
132 if not _SEARCH_PROPS_RE.match(line.strip()):
133 props.append(line)
134
135 # Append default search engine name
136 props.append("browser.search.defaultenginename=%s" % engine_order[0])
137
138 # Append search engine order
139 for i in range(0, min(3, len(engine_order))):
140 props.append("browser.search.order.%d=%s" % (i + 1, engine_order[i]))
141
142 # Write back list.txt
143 _write_lines(list_file_path, engine_ids)
144
145 # Write back region.properties
146 _write_lines(region_file_path, props)
147
148
149 class MinimalLogger:
150 def info(self, s):
151 print "INFO: %s" % s
152
153 def error(self, s):
154 print "ERROR: %s" % s
155
156 def fatal(self, s):
157 print "FATAL: %s" % s
158 exit(1)
159
160
161 def transform_locales(obj_dir, logger=MinimalLogger()):
162 chrome_path = os.path.join(obj_dir, "dist", "bin", "chrome")
163 if not os.path.exists(chrome_path):
164 logger.fatal("'%s' does not exist" % obj_dir)
165
166 locales = []
167 for p in next(os.walk(chrome_path))[1]:
168 if _LOCALE_RE.match(p):
169 locales.append(p)
170 locales.sort()
171
172 logger.info("Found %d locales" % len(locales))
173
174 for locale in locales:
175 locale_path = os.path.join(chrome_path, locale, "locale", locale)
176 if os.path.exists(locale_path):
177 _transform_locale(locale, locale_path, logger)
178 else:
179 logger.error("Missing 'locale' folder for '%s'" % locale)
180
LEFTRIGHT

Powered by Google App Engine
This is Rietveld