[jira] [Created] (KARAF-3964) cygpath call in "start" to convert CLASSPATH should first verify it's set

["David M. Karr (JIRA)" <ji...@apache.org>]

David M. Karr created KARAF-3964:
------------------------------------

             Summary: cygpath call in "start" to convert CLASSPATH should first verify it's set
                 Key: KARAF-3964
                 URL: https://issues.apache.org/jira/browse/KARAF-3964
             Project: Karaf
          Issue Type: Bug
          Components: karaf-core
         Environment: Windows 7, Cygwin
            Reporter: David M. Karr
            Priority: Trivial


Several of the Karaf scripts have "if $cygwin" blocks that convert some variables to Windows form. If the variable referenced is not set, cygpath will fail with "cygpath: can't convert empty path".  The impact is debatable, as the variable after the command will be unset, same as before it, but the error message can be confusing for users.

Most of these cases are likely dealt with, but one that is not is in the "start" script, for the "CLASSPATH" variable.  I don't set a global CLASSPATH value, and I believe it's generally a bad idea to set one.

The most reasonable fix is to simply not run cygpath if CLASSPATH is not set or is empty.  Therefore, change this:

CLASSPATH=`cygpath --path --windows "$CLASSPATH"`

to:

if [ ! -z "$CLASSPATH" ]; then
    CLASSPATH=`cygpath --path --windows "$CLASSPATH"`
fi




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)