imurchenko commented on pull request #8251:
URL: https://github.com/apache/ignite/pull/8251#issuecomment-694149573


   @vveider 
   Please check the output from shellcheck utility:
   
   In run.sh line 18:
   source "${IGNITE_HOME}"/bin/include/functions.sh
          ^-- SC1090: Can't follow non-constant source. Use a directive to 
specify location.
   
   
   In run.sh line 26:
   source "${IGNITE_HOME}"/bin/include/setenv.sh
          ^-- SC1090: Can't follow non-constant source. Use a directive to 
specify location.
   
   
   In run.sh line 33:
       IFS=, LIBS_LIST=("$(tr -d '[:space:]' <<< ${OPTION_LIBS})")
                                                 ^------------^ SC2086: Double 
quote to prevent globbing and word splitting.
   
   Did you mean:
       IFS=, LIBS_LIST=("$(tr -d '[:space:]' <<< "${OPTION_LIBS}")")
   
   
   In run.sh line 34:
     for lib in ${LIBS_LIST[@]}; do
   IGNITE-13453 Docker: Change run.sh to call java directly
                ^-------------^ SC2068: Double quote array expansions to avoid 
re-splitting elements.
   
   
   In run.sh line 71:
   if [ "${version}" -eq 8 ] ; then
         ^--------^ SC2154: version is referenced but not assigned.
   
   
   In run.sh line 98:
   if [ "${IGNITE_QUIET}" == "false" -o "${DIGNITE_QUIET}" == "false" ]; then
         ^-------------^ SC2153: Possible misspelling: IGNITE_QUIET may not be 
assigned, but DIGNITE_QUIET is.
                                     ^-- SC2166: Prefer [ p ] || [ q ] as [ p 
-o q ] is not well defined.
   
   
   In run.sh line 105:
     exec "${JAVA}" ${JVM_OPTS} -DIGNITE_HOME="${IGNITE_HOME}" -cp "${CP}" 
org.apache.ignite.startup.cmdline.CommandLineStartup "${DEFAULT_CONFIG}"
                    ^---------^ SC2086: Double quote to prevent globbing and 
word splitting.
   
   Did you mean:
     exec "${JAVA}" "${JVM_OPTS}" -DIGNITE_HOME="${IGNITE_HOME}" -cp "${CP}" 
org.apache.ignite.startup.cmdline.CommandLineStartup "${DEFAULT_CONFIG}"
   
   
   In run.sh line 107:
     exec "${JAVA}" ${JVM_OPTS} -DIGNITE_HOME="${IGNITE_HOME}" -cp "${CP}" 
org.apache.ignite.startup.cmdline.CommandLineStartup "${CONFIG_URI}"
                    ^---------^ SC2086: Double quote to prevent globbing and 
word splitting.
   
   Did you mean:
     exec "${JAVA}" "${JVM_OPTS}" -DIGNITE_HOME="${IGNITE_HOME}" -cp "${CP}" 
org.apache.ignite.startup.cmdline.CommandLineStartup "${CONFIG_URI}"


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to