Minor reworks to how script is structured for better efficiency

roles/backup/templates/restic-wrapper.sh

@@ -2,5 +2,4 @@
 exec nice -n 10 restic \
 	-r "s3:{{ backup_s3_aws_endpoint_url }}/{{ backup_s3_bucket }}/restic" \
 	-p /opt/restic-password \
-	--verbose \
 	"$@"

roles/backup/templates/s3backup.sh

@@ -48,16 +48,18 @@ fi
 # Helper functions
 backup() {
 	# Takes a file or directory to backup and backs it up
-	[ -z "$1" ] && return 1
+	[ -z "$@" ] && return 1
-
-	dir="$1"
-	echo "- $dir"
 
 	if command -v restic > /dev/null 2>&1; then
+		for dir in "$@"; do
+			echo "- $dir"
+		done
 		/opt/restic-wrapper \
 			backup \
-			"$dir"
+			"$@"
 	else
+		dir="$@"
+		echo "- $dir"
 		nice -n 10 tar {{ backup_s3backup_tar_args }}{{ backup_s3backup_tar_args_extra }} \
 	{% for item in backup_s3backup_exclude_list + backup_s3backup_exclude_list_extra %}
 			--exclude "{{ item }}" \
@@ -88,15 +90,19 @@ if [ -n "${DIRS[*]}" ]; then
 	echo "- {{ item }}"
 	{% endfor %}
 	echo "Will upload resultant backups to {{ backup_s3_bucket }}"
-	for dir in "${DIRS[@]}"; do
+	if command -v restic > /dev/null 2>&1; then
-		if [ "$dir" == "/data" ]; then
+		backup ${DIRS[*]}
-			for datadir in "$dir"/*; do
+	else
-				[ -e "$datadir" ] && backup "$datadir"
+		for dir in "${DIRS[@]}"; do
-			done
+			if [ "$dir" == "/data" ]; then
-		else
+				for datadir in "$dir"/*; do
-			backup "$dir"
+					[ -e "$datadir" ] && backup "$datadir"
-		fi
+				done
-	done
+			else
+				backup "$dir"
+			fi
+		done
+	fi
 fi
 
 # Dump Postgres DBs, if possible