fix: Assume exiftool path could contain whitespaces (#660)

* fix: Assume exiftool path could contain whitespaces * fix: Try to make mypy in github happy
mapillary · Sep 22, 2023 · 3bb46e6 · 3bb46e6
1 parent d492954
commit 3bb46e6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/mapillary_tools/upload.py b/mapillary_tools/upload.py
@@ -66,7 +66,7 @@ def wrap_http_exception(ex: requests.HTTPError):
     lines = [
         f"{ex.request.method} {resp.url}",
         f"> HTTP Status: {ex.response.status_code}",
-        f"{resp.content}",
+        str(resp.content),
     ]
     return UploadHTTPError("\n".join(lines))
 

diff --git a/mapillary_tools/video_data_extraction/extractors/exiftool_runtime_parser.py b/mapillary_tools/video_data_extraction/extractors/exiftool_runtime_parser.py
@@ -37,8 +37,17 @@ def __init__(
 
         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))
+        args = [
+            exiftool_path,
+            "-q",
+            "-r",
+            "-n",
+            "-ee",
+            "-api",
+            "LargeFileSupport=1",
+            "-X",
+            str(self.geotag_source_path),
+        ]
 
         xml_content = subprocess.run(args, capture_output=True, text=True).stdout