Browse Source

shellcheck: fix warnings in the hsaddress.sh script

Part of 30065.
teor 5 years ago
parent
commit
899d385c70
1 changed files with 6 additions and 5 deletions
  1. 6 5
      tools/hsaddress.sh

+ 6 - 5
tools/hsaddress.sh

@@ -7,12 +7,13 @@
 # Examples: tools/hsaddress.sh
 # Examples: tools/hsaddress.sh
 #           tools/hsaddress.sh 025h
 #           tools/hsaddress.sh 025h
 
 
-if [ ! -d "$CHUTNEY_PATH" -o ! -x "$CHUTNEY_PATH/chutney" ]; then
+if [ ! -d "$CHUTNEY_PATH" ] || [ ! -x "$CHUTNEY_PATH/chutney" ]; then
     # looks like a broken path: use the path to this tool instead
     # looks like a broken path: use the path to this tool instead
-    TOOLS_PATH=`dirname "$0"`
-    export CHUTNEY_PATH=`dirname "$TOOLS_PATH"`
+    TOOLS_PATH=$(dirname "$0")
+    CHUTNEY_PATH=$(dirname "$TOOLS_PATH")
+    export CHUTNEY_PATH
 fi
 fi
-if [ -d "$PWD/$CHUTNEY_PATH" -a -x "$PWD/$CHUTNEY_PATH/chutney" ]; then
+if [ -d "$PWD/$CHUTNEY_PATH" ] && [ -x "$PWD/$CHUTNEY_PATH/chutney" ]; then
     # looks like a relative path: make chutney path absolute
     # looks like a relative path: make chutney path absolute
     export CHUTNEY_PATH="$PWD/$CHUTNEY_PATH"
     export CHUTNEY_PATH="$PWD/$CHUTNEY_PATH"
 fi
 fi
@@ -48,7 +49,7 @@ then
     do
     do
         FILE="${dir}/$TARGET"
         FILE="${dir}/$TARGET"
         [ -e "$FILE" ] || continue
         [ -e "$FILE" ] || continue
-        echo -n "Node `basename ${dir}`: "
+        echo "Node $(basename "$dir"): " | tr -d "\n"
         show_address "$FILE"
         show_address "$FILE"
     done
     done
 elif [ $# -eq 1 ];
 elif [ $# -eq 1 ];