From: Gabriel Filion <gabster@lelutin.ca>
Date: Sun, 21 Jun 2015 03:51:15 +0000 (-0400)
Subject: import-rsnapshot: exit with 1, not -1
X-Git-Tag: 0.28-rc1~79
X-Git-Url: https://arthur.barton.de/gitweb/?a=commitdiff_plain;h=4f9cb846f3fc61816c498fe5f31cb8be2926c2db;hp=2660592aa67e987fb5ab8351ecea142faedc8e53;p=bup.git

import-rsnapshot: exit with 1, not -1

Using a negative exit code is a bashism and since we use /bin/sh to
interpret the script, we should avoid it.

This was reported by Raphael Geissert in a Debian bug report:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=772222

Signed-off-by: Gabriel Filion <gabster@lelutin.ca>
[rlb@defaultvalue.org: adjust commit message]
Reviewed-by: Rob Browning <rlb@defaultvalue.org>
Tested-by: Rob Browning <rlb@defaultvalue.org>
---

diff --git a/cmd/import-rsnapshot-cmd.sh b/cmd/import-rsnapshot-cmd.sh
index d04fbbb..6ab3984 100755
--- a/cmd/import-rsnapshot-cmd.sh
+++ b/cmd/import-rsnapshot-cmd.sh
@@ -5,7 +5,7 @@ usage() {
     echo "Usage: bup import-rsnapshot [-n]" \
         "<path to snapshot_root> [<backuptarget>]"
     echo "-n,--dry-run: just print what would be done"
-    exit -1
+    exit 1
 }
 
 DRY_RUN=