[PATCH 2/3] Fix common.config to allow SCRATCH_DEV and SCRATCH_MNT to be optional

Theodore Ts'o tytso at mit.edu
Fri Jun 19 12:53:32 CDT 2009


Commit 3ae9f2f8 purports to allow SCRATCH_DEV and SCRATCH_MNT to be
optional, but tests in common.config will cause check to exit with an
error if these environment variables are not defined.  Fix this problem.

Signed-off-by: "Theodore Ts'o" <tytso at mit.edu>
---
 common.config |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/common.config b/common.config
index 64df893..b304247 100644
--- a/common.config
+++ b/common.config
@@ -23,10 +23,10 @@
 #                   mounted XFS file system, needs to be be world
 #                   writeable
 # TEST_DEV -        device for file system containing TEST_DIR
-# SCRATCH_DEV -     device you can make a scratch file system on
-# SCRATCH_MNT -     mount point for scratch file system
 #
 # and optionally:
+# SCRATCH_DEV -     device you can make a scratch file system on
+# SCRATCH_MNT -     mount point for scratch file system
 # SCRATCH_LOGDEV -  scratch log device for external log testing
 # SCRATCH_RTDEV -   scratch rt dev
 # TEST_LOGDEV -     test log device for external log testing
@@ -202,8 +202,8 @@ known_hosts()
   [ -z "$TEST_DIR" ]       && MC="$MC TEST_DIR"
   [ -z "$TEST_DEV" ]       && MC="$MC TEST_DEV"
 
-  [ -z "$SCRATCH_MNT" ]    && MC="$MC SCRATCH_MNT"
-  [ -z "$SCRATCH_DEV" ]    && MC="$MC SCRATCH_DEV"
+#  [ -z "$SCRATCH_MNT" ]    && MC="$MC SCRATCH_MNT"
+#  [ -z "$SCRATCH_DEV" ]    && MC="$MC SCRATCH_DEV"
 
   [ "$MC" ] && (echo "Warning: need to define parameters for host $HOST";\
 	        echo "       or set variables:"; \
-- 
1.6.3.2.1.gb9f7d.dirty




More information about the xfs mailing list