fix: allow spaces in filenames for exiftool_runtime

mapillary · Sep 20, 2023 · ac933d0 · ptpt · Sep 20, 2023 · ac933d0
1 parent c43c482
commit ac933d0
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/mapillary_tools/video_data_extraction/extractors/exiftool_runtime_parser.py b/mapillary_tools/video_data_extraction/extractors/exiftool_runtime_parser.py
@@ -35,9 +35,11 @@ def __init__(
                 "Cannot execute exiftool. Please install it from https://exiftool.org/ or you package manager, or set the environment variable MAPILLARY_TOOLS_EXIFTOOL_PATH"
             )
 
-        args = (
-            f"{exiftool_path} -q -r -n -ee -api LargeFileSupport=1 -X {self.geotag_source_path}"
-        ).split(" ")
+        if not self.geotag_source_path:
+            return
+        args = f"{exiftool_path} -q -r -n -ee -api LargeFileSupport=1 -X".split(" ")
+        args.append(str(self.geotag_source_path))
+
         xml_content = subprocess.run(args, capture_output=True, text=True).stdout
 
         self.exiftoolXmlParser = ExiftoolXmlParser(