Skip to content

Commit ea2b2e2

Browse files
committed
Fix build_performance.sh to pass shellcheck
1 parent be4ec7e commit ea2b2e2

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tools/run_tests/performance/build_performance.sh

+5-5
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
source ~/.rvm/scripts/rvm
1717
set -ex
1818

19-
cd $(dirname $0)/../../..
19+
cd "$(dirname "$0")/../../.."
2020

2121
CONFIG=${CONFIG:-opt}
2222

@@ -28,11 +28,11 @@ then
2828
# TODO(jtattermusch): not embedding OpenSSL breaks the C# build because
2929
# grpc_csharp_ext needs OpenSSL embedded and some intermediate files from
3030
# this build will be reused.
31-
make CONFIG=${CONFIG} EMBED_OPENSSL=true EMBED_ZLIB=true qps_worker qps_json_driver -j8
31+
make CONFIG="${CONFIG}" EMBED_OPENSSL=true EMBED_ZLIB=true qps_worker qps_json_driver -j8
3232
fi
3333

3434
PHP_ALREADY_BUILT=""
35-
for language in $@
35+
for language in "$@"
3636
do
3737
case "$language" in
3838
"c++")
@@ -53,10 +53,10 @@ do
5353
fi
5454
;;
5555
"csharp")
56-
python tools/run_tests/run_tests.py -l $language -c $CONFIG --build_only -j 8 --compiler coreclr
56+
python tools/run_tests/run_tests.py -l "$language" -c "$CONFIG" --build_only -j 8 --compiler coreclr
5757
;;
5858
*)
59-
python tools/run_tests/run_tests.py -l $language -c $CONFIG --build_only -j 8
59+
python tools/run_tests/run_tests.py -l "$language" -c "$CONFIG" --build_only -j 8
6060
;;
6161
esac
6262
done

0 commit comments

Comments
 (0)