Message11229

Author jeff.allen
Recipients jamesmudd, jeff.allen
Date 2017-03-14.19:57:58
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1489521479.77.0.43169172747.issue2569@psf.upfronthosting.co.za>
In-reply-to
Content
I can certainly reproduce, but it works if I remove the quotes.

> set JAVA_HOME="C:\Program Files\Java\jdk1.8.0_121"

> jython --print
"\"C:\Program Files\Java\jdk1.8.0_121\"\bin\java" -Xmx512m -Xss1024k -classpath C:\jython\2.7.1rc1\jython.jar;. -Dpython.home=C:\jython\2.7.1rc1 -Dpython.executable=C:\jython\2.7.1rc1\bin\jython.exe -Dpython.launcher.uname=windows -Dpython.launcher.tty=true org.python.util.jython

Notice the questionable use of quotes? Tidy them up and it runs.

> "C:\Program Files\Java\jdk1.8.0_121\bin\java" -Xmx512m -Xss1024k -classpath C:\jython\2.7.1rc1\jython.jar;. -Dpython.home=C:\jython\2.7.1rc1 -Dpython.executable=C:\jython\2.7.1rc1\bin\jython.exe -Dpython.launcher.uname=windows -Dpython.launcher.tty=true org.python.util.jython
Jython 2.7.1rc1 (default:8a87a1e6d515, Feb 28 2017, 04:33:36)
[Java HotSpot(TM) 64-Bit Server VM (Oracle Corporation)] on java1.8.0_121
Type "help", "copyright", "credits" or "license" for more information.
>>> exit()

And similarly, if you define the variable without them:

> set JAVA_HOME=C:\Program Files\Java\jdk1.8.0_121

> jython
Jython 2.7.1rc1 (default:8a87a1e6d515, Feb 28 2017, 04:33:36)
[Java HotSpot(TM) 64-Bit Server VM (Oracle Corporation)] on java1.8.0_121
Type "help", "copyright", "credits" or "license" for more information.
>>>

I know it's quite common to quote paths in environment variables or scripts, especially if there are spaces invoilved, but this time it works against you. If we wanted to be resilient to this kind of thing (#2346), maybe we'd strip them in the launcher before composing the path?
History
Date User Action Args
2017-03-14 19:57:59jeff.allensetmessageid: <1489521479.77.0.43169172747.issue2569@psf.upfronthosting.co.za>
2017-03-14 19:57:59jeff.allensetrecipients: + jeff.allen, jamesmudd
2017-03-14 19:57:59jeff.allenlinkissue2569 messages
2017-03-14 19:57:58jeff.allencreate