Implement fn_expire_backups to reduce code duplication.

rsync_tmbackup.sh

@@ -122,6 +122,11 @@ if [ -f "$INPROGRESS_FILE" ]; then
 	fi
 fi
 
+fn_expire_backup() {
+    fn_log_info "Expiring $1"
+    echo rm -rf -- "$1"
+}
+
 # Run in a loop to handle the "No space left on device" logic.
 while [ "1" ]; do
 
@@ -165,11 +170,11 @@ while [ "1" ]; do
 
 		elif [ $stamp -ge $KEEP_DAILIES_DATE ]; then
 			# Delete all but the most recent of each day.
-			[ ${date:8:2} -eq ${prev:8:2} ] && echo rm -rf -- "$DEST_FOLDER/$date"
+			[ ${date:8:2} -eq ${prev:8:2} ] && fn_expire_backup "$fname"
 
 		else
 			# Delete all but the most recent of each month.
-			[ ${date:5:2} -eq ${prev:5:2} ] && echo rm -rf -- "$DEST_FOLDER/$date"
+			[ ${date:5:2} -eq ${prev:5:2} ] && fn_expire_backup "$fname"
 		fi
 
 		prev=$date
@@ -237,13 +242,13 @@ while [ "1" ]; do
 
 		fn_log_warn "No space left on device - removing oldest backup and resuming."
 
-		BACKUP_FOLDER_COUNT=$(find "$DEST_FOLDER" -type d -name "$BACKUP_FOLDER_PATTERN" -prune | wc -l)
+		BACKUP_FOLDER_COUNT=$(fn_find_backups | wc -l)
 		if [ "$BACKUP_FOLDER_COUNT" -lt "2" ]; then
 			fn_log_error "No space left on device, and no old backup to delete."
 			exit 1
 		fi
 
-		OLD_BACKUP_PATH=$(find "$DEST_FOLDER" -type d -name "$BACKUP_FOLDER_PATTERN" -prune | head -n 1)
+		OLD_BACKUP_PATH=$(fn_find_backups | head -n 1)
 		if [ "$OLD_BACKUP_PATH" == "" ]; then
 			fn_log_error "No space left on device, and cannot get path to oldest backup to delete."
 			exit 1
@@ -256,8 +261,7 @@ while [ "1" ]; do
 			exit 1
 		fi
 
-		fn_log_info "Deleting '$OLD_BACKUP_PATH'..."
+		fn_expire_backup "$OLD_BACKUP_PATH"
-		rm -rf -- "$OLD_BACKUP_PATH"
 
 		# Resume backup
 		continue