Clean up global variables
authorBryn M. Reeves <bmr@redhat.com>
Tue, 22 Jan 2013 17:55:17 +0000 (17:55 +0000)
committerBryn M. Reeves <bmr@redhat.com>
Tue, 22 Jan 2013 17:55:17 +0000 (17:55 +0000)
blkdevalias

index 7f4341f..7a2896f 100755 (executable)
@@ -20,6 +20,8 @@
 #  along with this program; if not, write to the Free Software
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 #
+
+# global associative arrays containing the device map
 declare -A WWIDMAP
 declare -A NAMEMAP
 declare -A TYPEMAP
@@ -27,18 +29,26 @@ declare -A PARTMAP
 
 # To enable debug output set the BA_DEBUG environment variable to "yes".
 #BA_DEBUG="yes"
+
+# defaults
 BA_DEFAULT_USER="oracle"
 BA_DEFAULT_GROUP="oracle"
 BA_DEFAULT_MODE="0660"
-BA_CONF_PATH="/etc/blkdevalias"
+
+# paths and configuration files
+BA_CONF_DIR="/etc/blkdevalias"
 BA_DEV_PATH="oracleasm/disks"
-BA_MAP_PATH="${BA_CONF_PATH}/blkdevalias.map"
-BA_CONF="${BA_CONF_PATH}/blkdevalias.conf"
+BA_MAP_PATH="${BA_CONF_DIR}/blkdevalias.map"
+BA_CONF_PATH="${BA_CONF_DIR}/blkdevalias.conf"
+BA_LOCK="/var/lock/blkdevalias"
+
+# user and group - only used if loading with a blank configuration file.
 BA_USER="$BA_DEFAULT_USER"
 BA_GROUP="$BA_DEFAULT_GROUP"
-BA_LOCK="/var/lock/blkdevalias"
-BA_SCSI_ID="/sbin/scsi_id"
+
+# programs used by blkdevalias
 BA_SCSI_ID_FLAGS="-g -u"
+BA_SCSI_ID="/sbin/scsi_id"
 BA_READLINK="/usr/bin/readlink"
 BA_BASENAME="/bin/basename"
 BA_DMSETUP="/sbin/dmsetup"
@@ -48,11 +58,13 @@ BA_SUDO="/usr/bin/sudo"
 BA_SED="/bin/sed"
 BA_ID="/usr/bin/id"
 BA_SH="/bin/bash"
+
+# real user and group ID the script is running as
 BA_REAL_UID="$($BA_ID -u)"
 BA_REAL_GID="$($BA_ID -g)"
 
 # load config
-. $BA_CONF
+. $BA_CONF_PATH
 
 _print () {
     echo -e "$@"
@@ -227,8 +239,8 @@ ba_chown_configs () {
     local _USER _GROUP _OUT
     _USER="$1"
     _GROUP="$2"
-    $BA_CHOWN -R "$_USER:$_GROUP" "$BA_CONF_PATH"
-    ls -l "$BA_CONF_PATH" | while read _OUT; do
+    $BA_CHOWN -R "$_USER:$_GROUP" "$BA_CONF_DIR"
+    ls -l "$BA_CONF_DIR" | while read _OUT; do
         _debug "ba_chown_configs: $_OUT"
     done
     return 0
@@ -322,7 +334,7 @@ EOF
     _debug "BA_USER=\"$BA_USER\""
     _debug "BA_GROUP=\"$BA_GROUP\""
     _print -n "Writing wwid map configuration: "
-    _TMP_CONF=$(mktemp --tmpdir=$BA_CONF_PATH)
+    _TMP_CONF=$(mktemp --tmpdir=$BA_CONF_DIR)
     if [ ! -f "$_TMP_CONF" ]; then
         _print
         _error "could not write temporary file"
@@ -333,9 +345,9 @@ EOF
     echo "BA_USER=\"$BA_USER\"" >> $_TMP_CONF
     echo "BA_GROUP=\"$BA_GROUP\"" >> $_TMP_CONF
     echo "BA_MODE=\"$BA_MODE\"" >> $_TMP_CONF
-    mv $_TMP_CONF $BA_CONF
+    mv $_TMP_CONF $BA_CONF_PATH
     if [ "$?" != 0 ]; then
-        _error "could not create configuration file $BA_CONF"
+        _error "could not create configuration file $BA_CONF_PATH"
         exit
     fi
     # set configration ownership to reflect admin users