[ 
https://issues.apache.org/jira/browse/TINKERPOP-2489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17924675#comment-17924675
 ] 

ASF GitHub Bot commented on TINKERPOP-2489:
-------------------------------------------

andreachild commented on code in PR #3020:
URL: https://github.com/apache/tinkerpop/pull/3020#discussion_r1945302491


##########
gremlin-server/src/main/bin/gremlin-server.sh:
##########
@@ -166,7 +166,7 @@ start() {
       exit 1
     fi
 
-    $JAVA -Dlogback.configurationFile=$LOGBACK_CONF $JAVA_OPTIONS -cp 
$CLASSPATH $GREMLIN_SERVER_CMD "$GREMLIN_YAML" >> "$LOG_FILE" 2>&1 &
+    $JAVA -Dlogback.configurationFile="$LOGBACK_CONF" $JAVA_OPTIONS -cp 
"$CLASSPATH" "$GREMLIN_SERVER_CMD" "$GREMLIN_YAML" >> "$LOG_FILE" 2>&1 &

Review Comment:
   Should `JAVA_OPTIONS` also be quoted? 





> Server doesn't start if folder has spaces
> -----------------------------------------
>
>                 Key: TINKERPOP-2489
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2489
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: server
>    Affects Versions: 3.4.8
>            Reporter: Divij Vaidya
>            Priority: Major
>
> Repro steps:
> 1. Download the server zip.
> 2. Unzip the binary.
> 3. Rename the unzipped folder and add a space, e.g. 
> {code:java}
> apache-tinkerpop-gremlin-server-3.4.9 my{code}
> 4. Start the server 
> {code:java}
> ./bin/gremlin-server.sh start{code}
> 5. The server will fail to start (check status) with the error "Error: Could 
> not find or load main class my.conf.log4j-server.properties"
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to