Avoid variable substitution at trap definition

This should fix the temporary files removal.
Revert last fix
2024-01-05 18:05:11 +01:00 · 2024-01-05 18:03:17 +01:00
1 changed files with 2 additions and 2 deletions
--- a/client/zzz_evobackup
+++ b/client/zzz_evobackup
@ -629,10 +629,10 @@ main() {

    # Initialize a list of files to delete at exit
    # Any file added to the list will also be deleted at exit
-    temp_files="${temp_files} ${PIDFILE}"
+    temp_files="${PIDFILE}"

    # shellcheck disable=SC2064
-    trap "rm -f ${temp_files}" EXIT
+    trap 'rm -f ${temp_files}' EXIT

    # Update canary to keep track of each run
    update-evobackup-canary --who "${PROGNAME}"
Author	SHA1	Message	Date
Alexis Ben Miloud--Josselin	e2c50f00d4	Avoid variable substitution at trap definition All checks were successful gitea/evobackup/pipeline/head This commit looks good Details This should fix the temporary files removal.	2024-01-05 18:05:11 +01:00
Alexis Ben Miloud--Josselin	50b5af6095	Revert last fix It breaks the backup. Because of `set -u`, we can initialize temp_files using temp_files.	2024-01-05 18:03:17 +01:00