Merge pull request #210 from eugenevd/20171014-remove_old_backups-find

bruno- · web-flow · commit 9c7ec631d4e5 · 2017-10-26T14:06:11.000+04:00
remove_old_backups() : replace xargs with -exec to handle 0 files found 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() {`