remove_old_backups() : replace xargs with -exec to handle 0 files found

Eugene Van Dam · Eugene Van Dam · commit 5f3873c502fe · 2017-10-25T14:30:50.000+02:00
by find better
diff --git a/scripts/save.sh b/scripts/save.sh
@@ -283,7 +283,7 @@ remove_old_backups() {
 	local -a files
 	files=($(ls -t $(resurrect_dir)/${RESURRECT_FILE_PREFIX}_*.${RESURRECT_FILE_EXTENSION} | tail -n +6))
 	[[ ${#files[@]} -eq 0 ]] ||
-		find "${files[@]}" -type f -mtime +30 | xargs rm
+		find "${files[@]}" -type f -mtime +30 -exec rm -v "{}" \;
 }
 
 save_all() {

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ remove_old_backups() {`
`283`	`283`	`local -a files`
`284`	`284`	`files=($(ls -t $(resurrect_dir)/${RESURRECT_FILE_PREFIX}_*.${RESURRECT_FILE_EXTENSION} \| tail -n +6))`
`285`	`285`	`[[ ${#files[@]} -eq 0 ]] \|\|`
`286`		`- find "${files[@]}" -type f -mtime +30 \| xargs rm`
	`286`	`+ find "${files[@]}" -type f -mtime +30 -exec rm -v "{}" \;`
`287`	`287`	`}`
`288`	`288`
`289`	`289`	`save_all() {`