Browse code

whitespace cleanup on functions & lib/config

fix some of the bash8 issues found in functions and lib/config,
part of the long march towards fixing all the bash8 issues.

Change-Id: Ia131f64870acb0f9d196fe1a9a45d633abb6fc4d

Sean Dague authored on 2013/10/22 20:43:22
Showing 2 changed files
... ...
@@ -1371,9 +1371,9 @@ function upload_image() {
1371 1371
         IMAGE="$FILES/${IMAGE_FNAME}"
1372 1372
         IMAGE_NAME="${IMAGE_FNAME%.xen-raw.tgz}"
1373 1373
         glance \
1374
-          --os-auth-token $token \
1375
-          --os-image-url http://$GLANCE_HOSTPORT \
1376
-          image-create \
1374
+            --os-auth-token $token \
1375
+            --os-image-url http://$GLANCE_HOSTPORT \
1376
+            image-create \
1377 1377
             --name "$IMAGE_NAME" --is-public=True \
1378 1378
             --container-format=tgz --disk-format=raw \
1379 1379
             --property vm_mode=xen < "${IMAGE}"
... ...
@@ -1396,11 +1396,11 @@ function upload_image() {
1396 1396
             mkdir "$xdir"
1397 1397
             tar -zxf $FILES/$IMAGE_FNAME -C "$xdir"
1398 1398
             KERNEL=$(for f in "$xdir/"*-vmlinuz* "$xdir/"aki-*/image; do
1399
-                     [ -f "$f" ] && echo "$f" && break; done; true)
1399
+                [ -f "$f" ] && echo "$f" && break; done; true)
1400 1400
             RAMDISK=$(for f in "$xdir/"*-initrd* "$xdir/"ari-*/image; do
1401
-                     [ -f "$f" ] && echo "$f" && break; done; true)
1401
+                [ -f "$f" ] && echo "$f" && break; done; true)
1402 1402
             IMAGE=$(for f in "$xdir/"*.img "$xdir/"ami-*/image; do
1403
-                     [ -f "$f" ] && echo "$f" && break; done; true)
1403
+                [ -f "$f" ] && echo "$f" && break; done; true)
1404 1404
             if [[ -z "$IMAGE_NAME" ]]; then
1405 1405
                 IMAGE_NAME=$(basename "$IMAGE" ".img")
1406 1406
             fi
... ...
@@ -1691,23 +1691,23 @@ function check_path_perm_sanity() {
1691 1691
 #
1692 1692
 # _vercmp_r sep ver1 ver2
1693 1693
 function _vercmp_r {
1694
-  typeset sep
1695
-  typeset -a ver1=() ver2=()
1696
-  sep=$1; shift
1697
-  ver1=("${@:1:sep}")
1698
-  ver2=("${@:sep+1}")
1694
+    typeset sep
1695
+    typeset -a ver1=() ver2=()
1696
+    sep=$1; shift
1697
+    ver1=("${@:1:sep}")
1698
+    ver2=("${@:sep+1}")
1699 1699
 
1700
-  if ((ver1 > ver2)); then
1701
-    echo 1; return 0
1702
-  elif ((ver2 > ver1)); then
1703
-    echo -1; return 0
1704
-  fi
1700
+    if ((ver1 > ver2)); then
1701
+        echo 1; return 0
1702
+    elif ((ver2 > ver1)); then
1703
+        echo -1; return 0
1704
+    fi
1705 1705
 
1706
-  if ((sep <= 1)); then
1707
-    echo 0; return 0
1708
-  fi
1706
+    if ((sep <= 1)); then
1707
+        echo 0; return 0
1708
+    fi
1709 1709
 
1710
-  _vercmp_r $((sep-1)) "${ver1[@]:1}" "${ver2[@]:1}"
1710
+    _vercmp_r $((sep-1)) "${ver1[@]:1}" "${ver2[@]:1}"
1711 1711
 }
1712 1712
 
1713 1713
 
... ...
@@ -1729,13 +1729,13 @@ function _vercmp_r {
1729 1729
 #
1730 1730
 # vercmp_numbers ver1 ver2
1731 1731
 vercmp_numbers() {
1732
-  typeset v1=$1 v2=$2 sep
1733
-  typeset -a ver1 ver2
1732
+    typeset v1=$1 v2=$2 sep
1733
+    typeset -a ver1 ver2
1734 1734
 
1735
-  IFS=. read -ra ver1 <<< "$v1"
1736
-  IFS=. read -ra ver2 <<< "$v2"
1735
+    IFS=. read -ra ver1 <<< "$v1"
1736
+    IFS=. read -ra ver2 <<< "$v2"
1737 1737
 
1738
-  _vercmp_r "${#ver1[@]}" "${ver1[@]}" "${ver2[@]}"
1738
+    _vercmp_r "${#ver1[@]}" "${ver1[@]}" "${ver2[@]}"
1739 1739
 }
1740 1740
 
1741 1741
 
... ...
@@ -10,7 +10,7 @@
10 10
 #   [[group-name|file-name]]
11 11
 #
12 12
 # group-name refers to the group of configuration file changes to be processed
13
-# at a particular time.  These are called phases in ``stack.sh`` but 
13
+# at a particular time.  These are called phases in ``stack.sh`` but
14 14
 # group here as these functions are not DevStack-specific.
15 15
 #
16 16
 # file-name is the destination of the config file
... ...
@@ -64,12 +64,12 @@ function get_meta_section_files() {
64 64
     [[ -r $file ]] || return 0
65 65
 
66 66
     $CONFIG_AWK_CMD -v matchgroup=$matchgroup '
67
-      /^\[\[.+\|.*\]\]/ {
68
-          gsub("[][]", "", $1);
69
-          split($1, a, "|");
70
-          if (a[1] == matchgroup)
71
-              print a[2]
72
-      }
67
+        /^\[\[.+\|.*\]\]/ {
68
+            gsub("[][]", "", $1);
69
+            split($1, a, "|");
70
+            if (a[1] == matchgroup)
71
+                print a[2]
72
+        }
73 73
     ' $file
74 74
 }
75 75