Clean up arg handling

net/remote/remote-client.sh

@@ -1,4 +1,4 @@
-#!/bin/bash -ex
+#!/bin/bash -e
 
 cd "$(dirname "$0")"/../..
 
@@ -7,9 +7,14 @@ entrypointIp="$2"
 numNodes="$3"
 RUST_LOG="$4"
 
-[[ -n $deployMethod ]] || exit
+missing() {
-[[ -n $entrypointIp ]] || exit
+  echo "Error: $1 not specified"
-[[ -n $numNodes ]] || exit
+  exit 1
+}
+
+[[ -n $deployMethod ]] || missing deployMethod
+[[ -n $entrypointIp ]] || missing entrypointIp
+[[ -n $numNodes ]]     || missing numNodes
 
 source net/common.sh
 loadConfigFile

net/remote/remote-node.sh

@@ -1,4 +1,4 @@
-#!/bin/bash -ex
+#!/bin/bash -e
 
 cd "$(dirname "$0")"/../..
 
@@ -9,11 +9,16 @@ entrypointIp="$4"
 numNodes="$5"
 RUST_LOG="$6"
 
-[[ -n $deployMethod ]] || exit
+missing() {
-[[ -n $nodeType ]] || exit
+  echo "Error: $1 not specified"
-[[ -n $publicNetwork ]] || exit
+  exit 1
-[[ -n $entrypointIp ]] || exit
+}
-[[ -n $numNodes ]] || exit
+
+[[ -n $deployMethod ]]  || missing deployMethod
+[[ -n $nodeType ]]      || missing nodeType
+[[ -n $publicNetwork ]] || missing publicNetwork
+[[ -n $entrypointIp ]]  || missing entrypointIp
+[[ -n $numNodes ]]      || missing numNodes
 
 cat > deployConfig <<EOF
 deployMethod="$deployMethod"

net/remote/remote-sanity.sh

@@ -1,4 +1,4 @@
-#!/bin/bash -ex
+#!/bin/bash -e
 #
 # This script is to be run on the leader node
 #
@@ -16,19 +16,15 @@ numNodes=
 # shellcheck source=/dev/null # deployConfig is written by remote-node.sh
 source deployConfig
 
-[[ -n $deployMethod ]] || {
+missing() {
-  echo deployMethod empty
+  echo "Error: $1 not specified"
-  exit 1
-}
-[[ -n $entrypointIp ]] || {
-  echo entrypointIp empty
-  exit 1
-}
-[[ -n $numNodes ]] || {
-  echo numNodes empty
   exit 1
 }
 
+[[ -n $deployMethod ]] || missing deployMethod
+[[ -n $entrypointIp ]] || missing entrypointIp
+[[ -n $numNodes ]]     || missing numNodes
+
 ledgerVerify=true
 validatorSanity=true
 while [[ $1 = -o ]]; do