Skip to content

Commit 15ce90e

Browse files
committed
fix deltion of multiple failed backup snapshots
fixes #6 regression introduced by shellcheck fixes (commit a42f661, PR #1). multiple snapshots were passed to `snapper delete` as one argument. revert this line and add shellcheck directive to silence error
1 parent 630e0ea commit 15ce90e

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

bin/snap-sync

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,9 @@ for x in $selected_configs; do
333333
fi
334334
done
335335
if [[ "$delete_failed" == [Yy]"es" || "$delete_failed" == [Yy] ]]; then
336-
snapper -c "$x" delete "$(snapper -c "$x" list --disable-used-space | awk '/'$name' backup in progress/ {print $1}')"
336+
# explicit split list of snapshots (on whitespace) into multiple arguments
337+
# shellcheck disable=SC2046
338+
snapper -c "$x" delete $(snapper -c "$x" list --disable-used-space | awk '/'$name' backup in progress/ {print $1}')
337339
fi
338340
fi
339341
fi

0 commit comments

Comments
 (0)