From e5cae33f5dbb3882700662a745dc9d4bdf529f16 Mon Sep 17 00:00:00 2001 From: Colin Cross Date: Thu, 11 Jul 2019 10:59:59 -0700 Subject: [PATCH] Allow jar wrapper to take quoted arguments Make the jar wrapper script correctly handle quoted arguments with spaces in them. Also allow JVM arguments in the form -J-XX, not just -JXX. Test: m checkbuild Change-Id: Iec5105bc390f2a12c6a4cda7f76d37585c39f520 --- scripts/jar-wrapper.sh | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/jar-wrapper.sh b/scripts/jar-wrapper.sh index 71c1d9067..b46804157 100644 --- a/scripts/jar-wrapper.sh +++ b/scripts/jar-wrapper.sh @@ -48,11 +48,11 @@ if [ ! -r "${jardir}/${jarfile}" ]; then exit 1 fi -javaOpts="" +declare -a javaOpts=() while expr "x$1" : 'x-J' >/dev/null; do - opt=`expr "$1" : '-J\(.*\)'` - javaOpts="${javaOpts} -${opt}" + opt=`expr "$1" : '-J-\{0,1\}\(.*\)'` + javaOpts+=("-${opt}") shift done -exec java ${javaOpts} -jar ${jardir}/${jarfile} "$@" +exec java "${javaOpts[@]}" -jar ${jardir}/${jarfile} "$@"