Fix shellcheck(1) warning SC2197

[backup-script.git] / bin / backup-script

diff --git a/bin/backup-script b/bin/backup-script
index a235190c3038a67f6941965e1099ee98443cfc24..f2df4db98220e0b12472249e53a883c0ff0db2d0 100755 (executable)
--- a/bin/backup-script
+++ b/bin/backup-script
@@ -16,6 +16,7 @@ PIDFILE="/var/run/$NAME.pid"
 DRYRUN=0
 VERBOSE=0
 TAG=""
+PREPOSTEXEC=1
 
 export LC_ALL=C
 
@@ -53,13 +54,14 @@ default_job_post_exec=""
 default_tags=""
 
 Usage() {
-       echo "Usage: $NAME [<options>] [<system> [<system> [...]]]"
+       echo "Usage: $NAME [<options>] [<job> [<job> [...]]]"
        echo
        echo "  -n, --dry-run       Test run only, don't copy any data."
        echo "  -p, --progress      Show progress, see rsync(1)."
        echo "  -t TAG, --tag TAG   Only run jobs with tag TAG."
+       echo "  -x, --no-exec       Don't run global pre-/post-exec commands."
        echo
-       echo "When no <system> is given, all defined systems are used."
+       echo "When no <job> is given, all defined systems are used."
        echo
        # shellcheck disable=SC2086
        echo -e $config_info
@@ -68,10 +70,9 @@ Usage() {
 }
 
 CleanUp() {
-       if [ -n "$post_exec" ]; then
+       if [[ -n "$pre_exec" && $PREPOSTEXEC -ne 0 ]]; then
                echo "Executing \"$post_exec\" ..."
-               sh -c $post_exec
-               if [ $? -ne 0 ]; then
+               if ! sh -c $post_exec; then
                        echo "Warning: post-exec command failed!"
                fi
                echo
@@ -305,6 +306,9 @@ while [ $# -gt 0 ]; do
                shift; TAG="$1"; shift
                [ -n "$TAG" ] || Usage
                ;;
+         "-x"|"--no-exec")
+               PREPOSTEXEC=0; shift
+               ;;
          "-"*)
                Usage
                ;;
@@ -317,13 +321,11 @@ echo -n "Started: "; date
 echo -e "$config_info"
 
 # Check rsync and its protocol version
-rsync=$(which "rsync" 2>/dev/null)
-if [ $? -ne 0 ]; then
+if ! rsync=$(which "rsync" 2>/dev/null); then
        echo "Failed to detect rsync(1)! Is it installed in your \$PATH?"
        exit 1
 fi
-rsync_proto=$($rsync --version 2>/dev/null | head -n 1 | sed 's/.*  protocol version \([0-9]*\)$/\1/')
-if [ $? -ne 0 ]; then
+if ! rsync_proto=$($rsync --version 2>/dev/null | head -n 1 | sed 's/.*  protocol version \([0-9]*\)$/\1/'); then
        echo "Failed to detect protocol version of $rsync!"
        exit 1
 fi
@@ -344,10 +346,9 @@ else
        sys=("${conf_d}/"*)
 fi
 
-if [ -n "$setup_exec" ]; then
+if [[ -n "$setup_exec" && $PREPOSTEXEC -ne 0 ]]; then
        echo "Executing \"$setup_exec\" ..."
-       sh -c $setup_exec
-       if [ $? -ne 0 ]; then
+       if ! sh -c $setup_exec; then
                echo "Error: setup command failed!"; echo
                echo "Aborting backup."; echo
                exit 5
@@ -367,18 +368,16 @@ if [ -e "$PIDFILE" ]; then
        echo
        exit 4
 fi
-touch "$PIDFILE" 2>/dev/null
-if [ $? -ne 0 ]; then
+if ! touch "$PIDFILE" 2>/dev/null; then
        echo "Warning: can't create PID file \"$PIDFILE\"!"
        echo
 else
        echo "$$" >>"$PIDFILE"
 fi
 
-if [ -n "$pre_exec" ]; then
+if [[ -n "$pre_exec" && $PREPOSTEXEC -ne 0 ]]; then
        echo "Executing \"$pre_exec\" ..."
-       sh -c $pre_exec
-       if [ $? -ne 0 ]; then
+       if ! sh -c $pre_exec; then
                echo "Error: pre-exec command failed!"; echo
                CleanUp
                echo "Aborting backup."; echo
@@ -459,8 +458,7 @@ for f in "${sys[@]}"; do
        [[ "$local" -eq 1 ]] && tags="$tags,LOCAL"
        # Check tags
        if [[ -n "$TAG" && "$TAG" != "ALL" ]]; then
-               echo "$tags" | grep -E "(^|,)$TAG(,|$)" >/dev/null 2>&1
-               if [ $? -ne 0 ]; then
+               if ! echo "$tags" | grep -E "(^|,)$TAG(,|$)" >/dev/null 2>&1; then
                        if [ "$DRYRUN" -ne 0 ]; then
                                echo "Tags of system \"$system\" don't match \"$TAG\": \"$tags\". Skipped."
                                echo
@@ -515,11 +513,11 @@ for f in "${sys[@]}"; do
        sys_root="$sys_target"
        if [[ "$DRYRUN" -eq 0 && ! -e "$sys_target" ]]; then
                if [ $generations -gt 0 ]; then
-                       CreateSubvolume "$sys_target"
+                       CreateSubvolume "$sys_target"; r=$?
                else
-                       mkdir -p "$sys_target"
+                       mkdir -p "$sys_target"; r=$?
                fi
-               if [ $? -ne 0 ]; then
+               if [ $r -ne 0 ]; then
                        echo "Can't create \"$sys_target\"!? \"$system\" skipped!"
                        echo; continue
                fi
@@ -527,8 +525,7 @@ for f in "${sys[@]}"; do
 
        if [[ "$local" -eq 0 && "$ping" -ne 0 ]]; then
                # Check if system is alive
-               ping -c 1 "$system" >/dev/null 2>&1
-               if [ $? -ne 0 ]; then
+               if ! ping -c 1 "$system" >/dev/null 2>&1; then
                        echo "Host \"$system\" seems not to be alive!? Skipped."
                        echo; continue
                fi
@@ -549,8 +546,7 @@ for f in "${sys[@]}"; do
                if [[ -n "$last" && ! -e "$last/.stamp" ]]; then
                        # Old backup directory without "stamp file", continue
                        echo "Found incomplete snapshot in \"$last\", reusing and renaming it ..."
-                       RenameSubvolume "$last" "$sys_target"
-                       if [ $? -ne 0 ]; then
+                       if ! RenameSubvolume "$last" "$sys_target"; then
                                echo "Failed to rename last snapshot \"$last\" to \"$sys_target\"!? \"$system\" skipped!"
                                echo; continue
                        fi
@@ -719,9 +715,8 @@ for f in "${sys[@]}"; do
                                # shellcheck disable=SC2086
                                echo "Removing backup from" $last "..."
                                if [ "$DRYRUN" -eq 0 ]; then
-                                       DeleteSubvolume "$dir"
-                                       [ $? -eq 0 ] || \
-                                         echo "Failed to delete \"$dir\"!"
+                                       DeleteSubvolume "$dir" \
+                                               || echo "Failed to delete \"$dir\"!"
                                fi
                        done
                        echo -n "Clean up finished: "; date