From 311c65737fd41a061fe3fe4b8822075e704f636c Mon Sep 17 00:00:00 2001 From: Adriaan de Groot Date: Fri, 11 Oct 2019 16:46:21 +0200 Subject: [PATCH] [unpackfs] Remove unused parameter - list_excludes() turns the extra mounts from global storage into --exclude parameters for rsync; it doesn't do anything with the destination parameter. - while here rename to something more descriptive --- src/modules/unpackfs/main.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/modules/unpackfs/main.py b/src/modules/unpackfs/main.py index a9e6ea067..035912d48 100644 --- a/src/modules/unpackfs/main.py +++ b/src/modules/unpackfs/main.py @@ -76,12 +76,9 @@ class UnpackEntry: ON_POSIX = 'posix' in sys.builtin_module_names -def list_excludes(destination): +def global_excludes(): """ List excludes for rsync. - - :param destination: - :return: """ lst = [] extra_mounts = globalstorage.value("extraMounts") @@ -120,7 +117,7 @@ def file_copy(source, dest, progress_cb): num_files_copied = 0 # Gets updated through rsync output args = ['rsync', '-aHAXr'] - args.extend(list_excludes(dest)) + args.extend(global_excludes()) args.extend(['--progress', source, dest]) process = subprocess.Popen( args, env=at_env, bufsize=1, stdout=subprocess.PIPE, close_fds=ON_POSIX