Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1951

gremlin-server.bat doesn't support paths containing spaces

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.3.2
    • Fix Version/s: 3.4.0, 3.3.4, 3.2.10
    • Component/s: server
    • Labels:
      None
    • Environment:
      Windows operating system

      Description

      If you copy the TinkerPop Gremlin Server to the folder

      C:\Program Files\Apache TinkerPop\Server
      

      then it would be impossible to start the server with the gremlin-server.bat file provided with the solution.

      The issue is that the spaces of the folder name will not be correctly handled when defining the CLASSPATH.

      It's possible to easily fix this issue by replacing the line

      java -Dlog4j.configuration=conf/log4j-server.properties %JAVA_OPTIONS% %JAVA_ARGS% -cp %LIBDIR%\*;%EXTDIR%; org.apache.tinkerpop.gremlin.server.GremlinServer %*
      

      by

      java -Dlog4j.configuration=conf/log4j-server.properties %JAVA_OPTIONS% %JAVA_ARGS% -cp "%LIBDIR%\*;%EXTDIR%;" org.apache.tinkerpop.gremlin.server.GremlinServer %*
      

      (the value provided for the -cp argument must be surrounded by double quotes).

      Same case a few lines bellow this occurrence with

      java -Dlog4j.configuration=conf/log4j-server.properties %JAVA_OPTIONS% %JAVA_ARGS% -cp %LIBDIR%\*;%EXTDIR%; org.apache.tinkerpop.gremlin.server.util.GremlinServerInstall %RESTVAR%
      

      to be replaced by

      java -Dlog4j.configuration=conf/log4j-server.properties %JAVA_OPTIONS% %JAVA_ARGS% -cp "%LIBDIR%\*;%EXTDIR%;" org.apache.tinkerpop.gremlin.server.util.GremlinServerInstall %RESTVAR%
      

       

        Attachments

          Activity

            People

            • Assignee:
              spmallette Stephen Mallette
              Reporter:
              Seddryck Cédric L. Charlier
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: