+1  :)

-Sundar

On Friday 05 June 2015 03:34 PM, Michael Haupt wrote:
Hi -

anyway, alternative webrev: 
http://cr.openjdk.java.net/~mhaupt/8080087/webrev.01 - please pick. :-)

Michael

Am 05.06.2015 um 11:57 schrieb Michael Haupt <michael.ha...@oracle.com>:

Hi Sundar,

I'm not tied to this solution. The code is as it is because it seems to be an 
issue of the Windows platform in particular. Would you think checking for key 
presence is  more robust?

Best,

Michael

Am 05.06.2015 um 11:49 schrieb A. Sundararajan 
<sundararajan.athijegannat...@oracle.com>:

Hi,

Can we do

if (! env.containsKey(ScriptingFunctions.PWD_NAME)) {
   env.put(ScriptingFunctions.PWD_NAME, System.getProperty("user.dir"), 
scriptEnv._strict);
}

in Global.java instead of checking for Windows platform explicitly?

-Sundar

On Friday 05 June 2015 03:08 PM, Michael Haupt wrote:
Dear all,

please review and sponsor this fix.
Bug: https://bugs.openjdk.java.net/browse/JDK-8080087
Webrev: http://cr.openjdk.java.net/~mhaupt/8080087/webrev.00

In Nashorn scripting mode, $ENV.PWD contains the current working directory. 
This is imported from the environment on unixy platforms. As the PWD variable 
is nonexistent on Windows by default, it needs to be set explicitly.

Thanks,

Michael


Reply via email to