Make sure either 'host' or 'remote' is set in rsync function

spraakbanken · Jun 28, 2022 · b26e8e8 · b26e8e8
1 parent 33a623c
commit b26e8e8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/sparv/api/util/install.py b/sparv/api/util/install.py
@@ -10,7 +10,7 @@
 logger = get_logger(__name__)
 
 
-def install_file(local_file, host, remote_file):
+def install_file(local_file, host=None, remote_file=None):
     """Rsync a file to a target host."""
     system.rsync(local_file, host, remote_file)
 

diff --git a/sparv/api/util/system.py b/sparv/api/util/system.py
@@ -153,6 +153,7 @@ def rsync(local, host=None, remote=None):
 
     When syncing directories, extraneous files in destination dirs are deleted.
     """
+    assert host or remote, "Either 'host' or 'remote' must be set."
     if remote is None:
         remote = local
     remote_dir = os.path.dirname(remote)