From ae2fac39c032ef9d4d327cdd7e1ed9ffe71ad619 Mon Sep 17 00:00:00 2001 From: a1346054 <36859588+a1346054@users.noreply.github.com> Date: Fri, 3 Sep 2021 13:48:55 +0000 Subject: [PATCH] fix shellcheck warnings --- make_release | 10 +++---- test_commands.sh | 70 ++++++++++++++++++++++++------------------------ 2 files changed, 40 insertions(+), 40 deletions(-) diff --git a/make_release b/make_release index 09c9374..f411663 100755 --- a/make_release +++ b/make_release @@ -10,7 +10,7 @@ tag=`awk '/IPERF_VERSION / { print $3 }' src/version.h` fi -dirname=`echo $tag $proj | awk '{ +dirname=`echo "$tag $proj" | awk '{ gsub(/-ALPHA/, "a", $1); gsub(/-BETA/, "b", $1); gsub(/-RELEASE/, "", $1); @@ -21,14 +21,14 @@ dirname=`echo $tag $proj | awk '{ do_tag () { - git tag -s -m "tagging $tag" $tag + git tag -s -m "tagging $tag" "$tag" } do_tar () { tarball=${dirname}.tar.gz - rm -f ${tarball} - git archive --format=tar --prefix ${dirname}/ ${tag} | gzip -9 > ${tarball} + rm -f "${tarball}" + git archive --format=tar --prefix "${dirname}/" "${tag}" | gzip -9 > "${tarball}" # Compute SHA256 hash case `uname -s` in @@ -37,7 +37,7 @@ do_tar () Darwin) sha="shasum -a 256" ;; *) sha=echo ;; esac - ${sha} ${tarball} | tee ${tarball}.sha256 + ${sha} "${tarball}" | tee "${tarball}.sha256" } usage () diff --git a/test_commands.sh b/test_commands.sh index 1cf2f43..abb9197 100755 --- a/test_commands.sh +++ b/test_commands.sh @@ -8,7 +8,7 @@ if [ $# -ne 1 ] then - echo "Usage: `basename $0` hostname" + echo "Usage: `basename "$0"` hostname" exit $E_BADARGS fi @@ -17,59 +17,59 @@ set -x host=$1 # basic testing -./src/iperf3 -c $host -V -t 5 -T "test1" -./src/iperf3 -c $host -u -V -t 5 +./src/iperf3 -c "$host" -V -t 5 -T "test1" +./src/iperf3 -c "$host" -u -V -t 5 # omit mode -./src/iperf3 -c $host -i .3 -O 2 -t 5 +./src/iperf3 -c "$host" -i .3 -O 2 -t 5 # JSON mode -./src/iperf3 -c $host -i 1 -J -t 5 +./src/iperf3 -c "$host" -i 1 -J -t 5 # force V4 -./src/iperf3 -c $host -4 -t 5 -./src/iperf3 -c $host -4 -u -t 5 +./src/iperf3 -c "$host" -4 -t 5 +./src/iperf3 -c "$host" -4 -u -t 5 # force V6 -./src/iperf3 -c $host -6 -t 5 -./src/iperf3 -c $host -6 -u -t 5 +./src/iperf3 -c "$host" -6 -t 5 +./src/iperf3 -c "$host" -6 -u -t 5 # FQ rate -./src/iperf3 -c $host -V -t 5 --fq-rate 5m -./src/iperf3 -c $host -u -V -t 5 --fq-rate 5m +./src/iperf3 -c "$host" -V -t 5 --fq-rate 5m +./src/iperf3 -c "$host" -u -V -t 5 --fq-rate 5m # SCTP -./src/iperf3 -c $host --sctp -V -t 5 +./src/iperf3 -c "$host" --sctp -V -t 5 # parallel streams -./src/iperf3 -c $host -P 3 -t 5 -./src/iperf3 -c $host -u -P 3 -t 5 +./src/iperf3 -c "$host" -P 3 -t 5 +./src/iperf3 -c "$host" -u -P 3 -t 5 # reverse mode -./src/iperf3 -c $host -P 2 -t 5 -R -./src/iperf3 -c $host -u -P 2 -t 5 -R +./src/iperf3 -c "$host" -P 2 -t 5 -R +./src/iperf3 -c "$host" -u -P 2 -t 5 -R # bidirectional mode -./src/iperf3 -c $host -P 2 -t 5 --bidir -./src/iperf3 -c $host -u -P 2 -t 5 --bidir +./src/iperf3 -c "$host" -P 2 -t 5 --bidir +./src/iperf3 -c "$host" -u -P 2 -t 5 --bidir # zero copy -./src/iperf3 -c $host -Z -t 5 -./src/iperf3 -c $host -Z -t 5 -R +./src/iperf3 -c "$host" -Z -t 5 +./src/iperf3 -c "$host" -Z -t 5 -R # window size -./src/iperf3 -c $host -t 5 -w 8M +./src/iperf3 -c "$host" -t 5 -w 8M # -n flag -./src/iperf3 -c $host -n 5M -./src/iperf3 -c $host -n 5M -u -b1G +./src/iperf3 -c "$host" -n 5M +./src/iperf3 -c "$host" -n 5M -u -b1G # -n flag with -R -./src/iperf3 -c $host -n 5M -R -./src/iperf3 -c $host -n 5M -u -b1G -R +./src/iperf3 -c "$host" -n 5M -R +./src/iperf3 -c "$host" -n 5M -u -b1G -R # conflicting -n -t flags -./src/iperf3 -c $host -n 5M -t 5 +./src/iperf3 -c "$host" -n 5M -t 5 # -k mode -./src/iperf3 -c $host -k 1K -./src/iperf3 -c $host -k 1K -u -b1G +./src/iperf3 -c "$host" -k 1K +./src/iperf3 -c "$host" -k 1K -u -b1G # -k mode with -R -./src/iperf3 -c $host -k 1K -R -./src/iperf3 -c $host -k 1K -u -b1G -R +./src/iperf3 -c "$host" -k 1K -R +./src/iperf3 -c "$host" -k 1K -u -b1G -R # CPU affinity -./src/iperf3 -c $host -A 2/2 -./src/iperf3 -c $host -A 2/2 -u -b1G +./src/iperf3 -c "$host" -A 2/2 +./src/iperf3 -c "$host" -A 2/2 -u -b1G # Burst mode -./src/iperf3 -c $host -u -b1G/100 +./src/iperf3 -c "$host" -u -b1G/100 # change MSS -./src/iperf3 -c $host -M 1000 -V +./src/iperf3 -c "$host" -M 1000 -V # test congestion control option (linux only) -./src/iperf3 -c $host -C reno -V +./src/iperf3 -c "$host" -C reno -V