replace urljoin with spack.util.url.join

GoogleCloudPlatform · Mar 20, 2024 · 8ab62ca · 8ab62ca
1 parent 974372f
commit 8ab62ca
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/ramble/ramble/cmd/deployment.py b/lib/ramble/ramble/cmd/deployment.py
@@ -13,6 +13,7 @@
 import llnl.util.filesystem as fs
 
 import spack.util.spack_json as sjson
+import spack.util.url as surl
 
 import ramble.cmd
 import ramble.cmd.common.arguments
@@ -106,9 +107,9 @@ def pull_file(src, dest):
 
     with ws.write_transaction():
         # Fetch deployment index first:
-        push_cls = ramble.pipeline.PublishDeploymentPipeline
+        push_cls = ramble.pipeline.PushDeploymentPipeline
 
-        remote_index_path = urllib.parse.urljoin(
+        remote_index_path = surl.join(
             args.deployment_path, ramble.pipeline.PushDeploymentPipeline.index_filename
         )
         local_index_path = \
@@ -120,7 +121,7 @@ def pull_file(src, dest):
             index_data = sjson.load(f)
 
         for file in index_data[push_cls.index_namespace]:
-            src = urllib.parse.urljoin(args.deployment_path, file)
+            src = surl.join(args.deployment_path, file)
             dest = os.path.join(ws.root, file)
             if os.path.exists(dest):
                 fs.force_remove(dest)