Clean var formatting in mod_for_factory

mod_for_factory_scripts/500populateQualDbs

Index: mod_for_factory_scripts/500populateQualDbs
diff --git a/mod_for_factory_scripts/500populateQualDbs b/mod_for_factory_scripts/500populateQualDbs
index fff9a05642290e05285b1fb554a9cc351d828fc0..2847a60389042a061954463f1a8c18da2739ca42 100755
--- a/mod_for_factory_scripts/500populateQualDbs
+++ b/mod_for_factory_scripts/500populateQualDbs
@@ -6,16 +6,18 @@
 
 TEST_DIR="${ROOT_FS_DIR}/usr/local/autotest/site_tests/hardware_Components"
 
-pushd ${TEST_DIR} 1> /dev/null
+if [ -d "${TEST_DIR}" ]; then
+  pushd "${TEST_DIR}" >/dev/null
 
-# Remove the DB directories belonging to other boards.
-KEEPDB="data_${BOARD}"
-ls -d data_* 2> /dev/null | grep -v ${KEEPDB} | xargs rm -fr
+  # Remove the DB directories belonging to other boards.
+  KEEPDB="data_${BOARD}"
+  ls -d data_* 2>/dev/null | grep -v "${KEEPDB}" | xargs rm -fr
 
-# Ensure there is DB directory in x86-agz and x86-mario.
-if [ ! -d ${KEEPDB} -a \
-    \( "${BOARD}" = "x86-agz" -o "${BOARD}" = "x86-mario" \) ]; then
-  echo "No component DB directory found at: ${KEEPDB}"
-fi
+  # Ensure there is DB directory in x86-agz and x86-mario.
+  if [ ! -d "${KEEPDB}" -a \

[Hung-Te, 2010/11/18 05:30:18]

I'd suggest to rewrite this as
if [ ! -d "${KEEPDB}" ] &&
   [ "${BOARD}" = "x86-agz" -o "{BOARD}" = "x86-mario" ]; then

which is more better for readability

[Nick Sanders, 2010/11/18 05:54:15]

Thanks, that is much better!

+      \( "${BOARD}" = "x86-agz" -o "${BOARD}" = "x86-mario" \) ]; then
+    echo "No component DB directory found at: ${KEEPDB}"
+  fi
 
-popd 1> /dev/null
+  popd >/dev/null
+fi