Skip to content

Commit

Permalink
IGNITE-23739 CLI doesn't accept options with spaces (apache#4770)
Browse files Browse the repository at this point in the history
  • Loading branch information
valepakh authored Nov 21, 2024
1 parent ee004ae commit 887a3da
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 11 deletions.
1 change: 0 additions & 1 deletion packaging/cli/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,6 @@ def tokens = [
APP_JAR : "${project(':ignite-cli').name}-${project(':ignite-cli').version}.jar".toString(),
SETUP_JAVA_FILE_NAME: 'setup-java.sh',
MAIN_CLASS : 'org.apache.ignite.internal.cli.Main',
ARGS : '$@'
]

def createChecksums = tasks.register('createChecksums', Checksum) {
Expand Down
11 changes: 1 addition & 10 deletions packaging/cli/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,19 +22,10 @@
LOG_DIR="@LOG_DIR@"
CLASSPATH="@LIB_DIR@/@APP_JAR@:@LIB_DIR@/*"
MAIN_CLASS="@MAIN_CLASS@"
ARGS="@ARGS@"

DEFAULT_JVM_OPTS="-Dfile.encoding=UTF-8 \
-XX:+HeapDumpOnOutOfMemoryError \
-XX:+ExitOnOutOfMemoryError \
-XX:HeapDumpPath=${LOG_DIR}"


CMD="${JAVACMD} \
${DEFAULT_JVM_OPTS} \
-classpath ${CLASSPATH} \
${MAIN_CLASS} \
${ARGS}"

${CMD}

${JAVACMD} ${DEFAULT_JVM_OPTS} -classpath ${CLASSPATH} ${MAIN_CLASS} "$@"

0 comments on commit 887a3da

Please sign in to comment.