source and destinationPath can change during post processes

Author: typemytype

Batch.roboFontExt/lib/batchGenerators/batchTools.py

@@ -156,6 +156,7 @@ def __call__(self, sourcePath, destinationPath):
             newSourceDestinationPath = callback(sourcePath, destinationPath)
             if newSourceDestinationPath:
                 sourcePath, destinationPath = newSourceDestinationPath
+        return sourcePath, destinationPath
 
     def __del__(self):
         self.callbacks = None
@@ -257,11 +258,11 @@ def generatePaths(
             )
             sourcePath = os.path.join(fontDir, tempFileName)
             destinationPath = os.path.join(fontDir, fileName)
-            postProcessCallback(
+            sourcePath, destinationPath = postProcessCallback(
                 sourcePath,
                 destinationPath
             )
-            if os.path.exists(sourcePath):
+            if os.path.exists(sourcePath) and sourcePath != destinationPath:
                 shutil.copyfile(sourcePath, destinationPath)
                 os.remove(sourcePath)
 

Batch.roboFontExt/lib/batchGenerators/variableFontsGenerator/__init__.py

@@ -429,11 +429,11 @@ def build(root, generateOptions, settings, progress, report):
 
                 sourcePath = os.path.join(fontDir, tempFileName)
                 destinationPath = os.path.join(fontDir, fileName)
-                postProcessCallback(
+                sourcePath, destinationPath = postProcessCallback(
                     sourcePath,
                     destinationPath
                 )
-                if os.path.exists(sourcePath):
+                if os.path.exists(sourcePath) and sourcePath != destinationPath:
                     shutil.copyfile(sourcePath, destinationPath)
                     os.remove(sourcePath)
 

Batch.roboFontExt/lib/batchGenerators/webFontsGenerator/__init__.py

@@ -17,7 +17,8 @@
 cssFormatExtMap = {
     ".otf": "opentype",
     ".ttf": "truetype",
-    ".woff": "woff"
+    ".woff": "woff",
+    ".woff2": "woff2"
 
 }