Issue 1144 - Generate IE update manifests

sitescripts/extensions/bin/createNightlies.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2014 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 13 matching lines @@
   with changelogs and documentation.
 
 """
 
 import sys, os, os.path, subprocess, ConfigParser, traceback, json, hashlib
 import tempfile, shutil, urlparse, pipes, time, urllib2, struct
 from datetime import datetime
 from urllib import urlencode
 from xml.dom.minidom import parse as parseXml
 from sitescripts.utils import get_config, setupStderr, get_template
-from sitescripts.extensions.utils import (
-  compareVersions, Configuration, getSafariCertificateID, writeIEUpdateManifest)
+from sitescripts.extensions.utils import compareVersions, Configuration, getSafariCertificateID
 
 MAX_BUILDS = 50
 
 
 class NightlyBuild(object):
   """
     Performs the build process for an extension,
     generating changelogs and documentation.
   """
 
@@ skipping 167 matching lines @@
       Writes update.json file for the latest IE build
     """
     if len(versions) == 0:
       return
 
     version = versions[0]
     packageName = self.basename + '-' + version + self.config.packageSuffix
     updateURL = urlparse.urljoin(self.config.nightliesURL, self.basename + '/' + packageName + '?update')
     baseDir = os.path.join(self.config.nightliesDirectory, self.basename)
     manifestPath = os.path.join(baseDir, 'update.json')
-    writeIEUpdateManifest(manifestPath, [{

[Felix Dahlke, 2014/07/29 15:10:27]

Not brilliant that this function has the same name as the current method -
suggestions?

[Wladimir Palant, 2014/07/30 14:02:23]

Don't import the function globally but only in this function, under an alias?

from sitescripts.extensions.utils import writeIEUpdateManifest as doWrite

doWrite(...)

[Felix Dahlke, 2014/07/30 14:22:36]

Yeah, I like that. Done.

+
+    from sitescripts.extensions.utils import writeIEUpdateManifest as doWrite
+    doWrite(manifestPath, [{
       'basename': self.basename,
       'version': version,
       'updateURL': updateURL
     }])
 
     for suffix in (self.config.packageSuffix, self.config.packageSuffix.replace("-x64", "-x86")):
       linkPath = os.path.join(baseDir, '00latest%s' % suffix)
       outputPath = os.path.join(baseDir, self.basename + '-' + version + suffix)
       if hasattr(os, 'symlink'):
         if os.path.exists(linkPath):
@@ skipping 257 matching lines @@
     except Exception, ex:
       print >>sys.stderr, "The build for %s failed:" % repo
       traceback.print_exc()
 
   file = open(nightlyConfigFile, 'wb')
   nightlyConfig.write(file)
 
 
 if __name__ == '__main__':
   main()