More build tools improvements

packagerGecko.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus build tools,
 # Copyright (C) 2006-2012 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 107 matching lines @@
 
   for locale in locales:
     data = SafeConfigParser()
     data.optionxform = str
     try:
       result[locale] = localeTools.readFile(os.path.join(getLocalesDir(baseDir), locale, 'meta.properties'))
     except:
       result[locale] = {}
   return result
 
-def getContributors(baseDir, metadata):
+def getContributors(metadata):
   main = []
   additional = set()
   if metadata.has_section('contributors'):
     options = metadata.options('contributors')
     options.sort()
     for option in options:
       value = metadata.get('contributors', option)
       if re.search(r'\D', option):
         match = re.search(r'^\s*(\S+)\s+//([^/\s]+)/@(\S+)\s*$', value)
         if not match:
           print >>sys.stderr, 'Warning: unrecognized contributor location "%s"\n' % value
           continue
-        dom = minidom.parse(os.path.join(baseDir, match.group(1)))
+        baseDir = os.path.dirname(metadata.option_source('contributors', option))
+        parts = match.group(1).split('/')
+        dom = minidom.parse(os.path.join(baseDir, *parts))
         tags = dom.getElementsByTagName(match.group(2))
         for tag in tags:
           if tag.hasAttribute(match.group(3)):
             for name in re.split(r'\s*,\s*', tag.getAttribute(match.group(3))):
               additional.add(name)
       else:
         main.append(value)
   return main + sorted(additional, key=unicode.lower)
 
 def initTranslators(localeMetadata):
@@ skipping 147 matching lines @@
     locales = getLocales(baseDir)
   elif locales == 'all':
     locales = getLocales(baseDir, True)
 
   metadata = readMetadata(baseDir)
   version = getBuildVersion(baseDir, metadata, releaseBuild, buildNum)
 
   if outFile == None:
     outFile = getDefaultFileName(baseDir, metadata, version, 'xpi')
 
-  contributors = getContributors(baseDir, metadata)
+  contributors = getContributors(metadata)
 
   params = {
     'baseDir': baseDir,
     'locales': locales,
     'releaseBuild': releaseBuild,
     'version': version.encode('utf-8'),
     'metadata': metadata,
     'contributors': contributors,
     'multicompartment': multicompartment,
   }
 
   files = Files(getPackageFiles(params), getIgnoredFiles(params),
                 process=lambda path, data: processFile(path, data, params))
   files['install.rdf'] = createManifest(params)
   if metadata.has_section('mapping'):
-    files.readMappedFiles(baseDir, metadata.items('mapping'))
+    files.readMappedFiles(metadata.items('mapping'))
   files.read(baseDir, skip=('chrome'))
   for name, path in getChromeSubdirs(baseDir, params['locales']).iteritems():
     if os.path.isdir(path):
       files.read(path, 'chrome/%s' % name)
   fixupLocales(params, files)
   if not 'bootstrap.js' in files:
     addMissingFiles(params, files)
   if keyFile:
     signFiles(files, keyFile)
   files.zip(outFile, sortKey=lambda x: '!' if x == 'META-INF/zigbert.rsa' else x)
 
 def autoInstall(baseDir, host, port, multicompartment=False):
   fileBuffer = StringIO()
   createBuild(baseDir, outFile=fileBuffer, multicompartment=multicompartment)
   urllib.urlopen('http://%s:%s/' % (host, port), data=fileBuffer.getvalue())