[tor-commits] [tor/master] Update test-network.sh to bash script to use array

nickm at torproject.org nickm at torproject.org
Tue Mar 5 16:53:37 UTC 2019


commit f888b3e2ee23f6e0394c9c7589c10e320714deda
Author: rl1987 <rl1987 at sdf.lonestar.org>
Date:   Sat Feb 2 16:46:30 2019 +0200

    Update test-network.sh to bash script to use array
    
    This lets us to save original script argument to array (POSIX shell does not
    support that). Fixes shellcheck warnings SC2124 and SC2086.
---
 src/test/test-network.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/test/test-network.sh b/src/test/test-network.sh
index 51b298ca8..e382eec66 100755
--- a/src/test/test-network.sh
+++ b/src/test/test-network.sh
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 
 # This script calls the equivalent script in chutney/tools
 
@@ -20,7 +20,7 @@ myname=$(basename $0)
 
 # Save the arguments before we destroy them
 # This might not preserve arguments with spaces in them
-ORIGINAL_ARGS="$@"
+ORIGINAL_ARGS=( "$@" )
 
 # We need to find CHUTNEY_PATH, so that we can call the version of this script
 # in chutney/tools with the same arguments. We also need to respect --quiet.
@@ -99,7 +99,7 @@ if [ -d "$CHUTNEY_PATH" ] && [ -x "$TEST_NETWORK" ]; then
     # this may fail if some arguments have spaces in them
     # if so, set CHUTNEY_PATH before calling test-network.sh, and spaces
     # will be handled correctly
-    exec "$TEST_NETWORK" $ORIGINAL_ARGS
+    exec "$TEST_NETWORK" "${ORIGINAL_ARGS[@]}" # $ORIGINAL_ARGS
 else
     $ECHO "$myname: Could not find tools/test-network.sh in CHUTNEY_PATH."
     $ECHO "$myname: Please update your chutney using 'git pull'."





More information about the tor-commits mailing list