From: Micah Anderson <micah@riseup.net>
Date: Sat, 24 Nov 2007 23:13:24 +0000 (+0000)
Subject: fix Trac#25
X-Git-Tag: backupninja-0.9.5~11
X-Git-Url: https://git.stderr.nl/gitweb?p=matthijs%2Fupstream%2Fbackupninja.git;a=commitdiff_plain;h=84a6d7fa7a6c2ebd4e2b6687dba8fc6addfb843a

fix Trac#25
---

diff --git a/ChangeLog b/ChangeLog
index 50a4652..6ebf602 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -30,6 +30,7 @@ version 0.9.5 -- unreleased
 	 . Added an examples file (Closes: Trac#23)
 	 . Applied patch from Anarcat that fixes the cp/mkdir calls to not use GNU 
 	   coreutils options, as well as some bashisms (Closes: Trac#24)
+	 . Fix test mode (Closes: Trac#25)
         mysql:
 	 . Fixed case where odd combination of configuration options caused sqldump
 	   backups to get overwritten with an empty file (Closes: #402679)
diff --git a/handlers/maildir.in b/handlers/maildir.in
index 0d88e92..7a68e85 100644
--- a/handlers/maildir.in
+++ b/handlers/maildir.in
@@ -62,7 +62,10 @@ srcdir=${srcdir%/}
 
 [ "$multiconnection" == "notset" ] && fatal "The maildir handler uses a very different destination format. See the example .maildir for more information"
 
-[ ! $test ] || testflags="--dry-run -v"
+if [ $test ]; then
+   testflags="--dry-run -v"
+fi
+
 rsyncflags="$testflags -e 'ssh -p $destport' -r -v --ignore-existing --delete --size-only --bwlimit=$speedlimit"
 excludes="--exclude '.Trash/\*' --exclude '.Mistakes/\*' --exclude '.Spam/\*'"