Prepared buildtools for Safari

packagerSafari.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus build tools,
 # Copyright (C) 2006-2013 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 101 matching lines @@
     ).split()
   ).encode('utf-8')
 
 def createInfoModule(params):
   template = getTemplate('safariInfo.js.tmpl')
   return template.render(params).encode('utf-8')
 
 def fixAbsoluteUrls(files):
   for filename, content in files.iteritems():
     if os.path.splitext(filename)[1].lower() == '.html':
-      prefix = '/'.join(['..'] * filename.count('/') + [''])
       files[filename] = re.sub(
-        r'(<.*?\b(?:href|src)\s*=\s*(["\']))(\/.*?)(?=\2.*?>)',
-        lambda m: '%s%s%s' % (m.group(1), prefix, m.group(3).lstrip('/')),

[Wladimir Palant, 2013/10/31 13:58:19]

Why look for the entire tag if all you need is the leading slash inside the
attribute? Also, if the intention was to match href="..." only inside a tag then
it didn't work - this regexp will happily match <>href="/fo"<>. How about:

    r'(<[^<>]*?\b(?:href|src)\s*=\s*(?:["\']))\/',
    r'\1%s' % prefix,

+        r'(<[^<>]*?\b(?:href|src)\s*=\s*["\']?)\/+',
+        r'\1' + '/'.join(['..'] * filename.count('/') + ['']),
         content, re.S | re.I
       )
 
 def createSignedXarArchive(outFile, files, keyFile):
   import subprocess
   import tempfile
   import shutil
   import M2Crypto
 
   # write files to temporary directory and create a xar archive
@@ skipping 125 matching lines @@
   fixAbsoluteUrls(files)
 
   dirname = metadata.get('general', 'basename') + '.safariextension'
   for filename in files.keys():
     files[os.path.join(dirname, filename)] = files.pop(filename)
 
   if keyFile:
     createSignedXarArchive(outFile, files, keyFile)
   else:
     files.zip(outFile)