]> diplodocus.org Git - nmh/commitdiff
test/common.sh.in: Only rm(1) if the cd(1) succeeds.
authorRalph Corderoy <ralph@inputplus.co.uk>
Sun, 24 Sep 2017 12:12:46 +0000 (13:12 +0100)
committerRalph Corderoy <ralph@inputplus.co.uk>
Sun, 24 Sep 2017 12:12:46 +0000 (13:12 +0100)
The rm specified the full path anyway, but if the cd fails for some odd
reason then it's best not to continue.

test/common.sh.in

index 1dbbf2c..e866033 100644 (file)
@@ -302,7 +302,7 @@ cleanup() {
     #### cd to $MH_TEST_DIR before trying to remove its Mail
     #### subdirectory.  rm on Solaris won't remove it if it's in the
     #### path of the current working directory.
-    test -z "$MH_TEST_NOCLEANUP" && (cd "$MH_TEST_DIR"; rm -rf "$MH_TEST_DIR"/Mail)
+    test -z "$MH_TEST_NOCLEANUP" && (cd "$MH_TEST_DIR" && rm -rf "$MH_TEST_DIR"/Mail)
 
     #### Report test name if set, which indicates failure.
     #### Relies on set -e to invoke the trap which calls