Issue 3498 - Improve file mapping / skipping logic

packager.py

Index: packager.py
diff --git a/packager.py b/packager.py
index 71ac371faf5f11b623c99ff9ed3aebbc2f8ce747..609b1ebf82a525084a9bfd156b7ddbdfef060ea7 100644
--- a/packager.py
+++ b/packager.py
@@ -88,18 +88,17 @@ class Files(dict):
         return False
     return True
 
-  def read(self, path, relpath='', skip=None):
+  def read(self, path, relpath='', skip=()):
     if os.path.isdir(path):
       for file in os.listdir(path):
         name = relpath + ('/' if relpath != '' else '') + file
-        if (skip == None or file not in skip) and self.isIncluded(name):
-          self.read(os.path.join(path, file), name)
+        if (name not in skip) and self.isIncluded(name):
+          self.read(os.path.join(path, file), name, skip)
     else:
       with open(path, 'rb') as file:
         if relpath in self:
           print >>sys.stderr, 'Warning: File %s defined multiple times' % relpath
-        else:
-          self[relpath] = file.read()
+        self[relpath] = file.read()
 
   def readMappedFiles(self, mappings):
     for item in mappings: