Prepared buildtools for Safari

packagerChrome.py

Index: packagerChrome.py
===================================================================
--- a/packagerChrome.py
+++ b/packagerChrome.py
@@ -160,6 +160,23 @@
     sourceFiles = map(lambda f: os.path.abspath(os.path.join(baseDir, f)), sourceFiles)
     files[file] = doRewrite(sourceFiles, args)
 
+def convertFirstRunPage(params, files):

[Felix Dahlke, 2013/10/15 16:24:38]

As I said in the other review, I think it'd be fine to just always use a
template for the first run page.

[Sebastian Noack, 2013/10/15 17:06:56]

My intention was to don't make the change on the first run page depend on the
change on the buildtools and vice versa. So the firstRun.html will just work
like before, without processing it with jinja2.

In my experience it is always a mess to require two repositories that depend on
each other, to be updated at the same time. For example the changes on the
Firefox extension, would have to update the revision of the buildtools the
subrepo reffers to, to a version that doesn't exist yet. We also would have to
update the Chrome repo at the time we merge this (don't know about IE), if we
require the first run page to be always generated.

What do you think? I think it would be better to do it in two steps, if we want
the first run page to be always generated.

[Felix Dahlke, 2013/10/15 22:46:11]

Fair enough, sure.

+  import jinja2
+
+  env = jinja2.Environment(
+    block_start_string='<!-- {%',
+    block_end_string='%} -->',
+    variable_start_string='<!-- {{',
+    variable_end_string='}} -->',
+    comment_start_string='<-- {#',
+    comment_end_string='#} -->',
+
+	trim_blocks=True,

[Felix Dahlke, 2013/10/15 16:24:38]

Seems like the indentation is off here.

[Sebastian Noack, 2013/10/15 17:06:56]

True, I'll fix that.

+	lstrip_blocks=True
+  )
+
+  files['firstRun.html'] = env.from_string(files['firstRun.html']).render(type=params['type'])
+
 def importGeckoLocales(params, files):
   import localeTools
 
@@ -279,10 +296,11 @@
           files[operaFile] = files[chromeFile]
         del files[chromeFile]
 
-    # Hack: Replace "Chrome" by "Opera" in the locales
+  if params['type'] in ('opera', 'safari'):
+    # Hack: Replace "Chrome" by "Opera" or "Safari" in the locales
     for path, data in files.iteritems():
       if path.startswith("_locales/") and path.endswith("/messages.json"):
-        files[path] = re.sub(r"\bChrome\b", "Opera", data)
+        files[path] = re.sub(r"\bChrome\b", params['type'].capitalize(), data)
 
 def signBinary(zipdata, keyFile):
   import M2Crypto
@@ -335,6 +353,13 @@
   if metadata.has_section('convert_js'):
     convertJS(params, files)
 
+  if metadata.has_section('convert_img'):
+    from imageConversion import convertImages
+    convertImages(params, files)
+
+  if 'firstRun.html' in files:
+    convertFirstRunPage(params, files)
+
   if metadata.has_section('import_locales'):
     importGeckoLocales(params, files)