Use a list in verify_aip to execute atool command

Closes #449.
artefactual-labs · Feb 9, 2024 · 3d22c85 · 3d22c85
1 parent a1d4f6e
commit 3d22c85
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -21,9 +21,15 @@ See the fragment files in the `changelog.d directory`_.
 
 .. _changelog-0.7.7:
 
-0.7.7 — 2024-02-08
+0.7.7 — 2024-02-09
 ==================
 
+Fixed
+-----
+
+- Pass arguments to atool using a list, allowing transfer names to contain
+  spaces. See issue #449 for more details.
+
 Changed
 -------
 

diff --git a/a3m/client/clientScripts/verify_aip.py b/a3m/client/clientScripts/verify_aip.py
@@ -14,7 +14,6 @@
 from a3m.executeOrRunSubProcess import executeOrRun
 from a3m.main.models import File
 
-
 logger = logging.getLogger(__name__)
 
 
@@ -24,7 +23,7 @@ class VerifyChecksumsError(Exception):
 
 def extract_aip(job, aip_path, extract_path):
     os.makedirs(extract_path)
-    command = f"atool --extract-to={extract_path} -V0 {aip_path}"
+    command = ["atool", f"--extract-to={extract_path}", "-V0", aip_path]
     job.pyprint("Running extraction command:", command)
     exit_code, stdout, stderr = executeOrRun("command", command, capture_output=True)
     job.write_output(stdout)