Hi everyone,
    
                                                                                  This small fix concerns             jdk.testlibrary.JdkFinder utility class. The         check below is too OS dependent and should be deleted.         
        ...         
                binPath += File.separatorChar + "bin" +         File.separatorChar + executable;         
                File toolFile = new File(binPath);         
        ==> if (!toolFile.exists()) {         
                    throw new RuntimeException(binPath + " does not         exist");         
                }         
        ...         
        
        On windows if executable is specified as e.g. 'java', without         '.exe', the File exists() will fail. But it's still possible to         run executable.
        
                
    Webrev:
    http://cr.openjdk.java.net/~ykantser/8008089/webrev.00/
    
    Bug:
    http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8008089
    
    Testing:
    JPRT
    
    Thanks,
    Katja
   ">

(original) (raw)

Hi everyone,

This small fix concerns jdk.testlibrary.JdkFinder utility class. The check below is too OS dependent and should be deleted.
...
        binPath += File.separatorChar + "bin" + File.separatorChar + executable;
        File toolFile = new File(binPath);
\==> if (!toolFile.exists()) {
            throw new RuntimeException(binPath + " does not exist");
        }
...

On windows if executable is specified as e.g. 'java', without '.exe', the File exists() will fail. But it's still possible to run executable.


Webrev:
http://cr.openjdk.java.net/\~ykantser/8008089/webrev.00/

Bug:
http://bugs.sun.com/bugdatabase/view\_bug.do?bug\_id=8008089

Testing:
JPRT

Thanks,
Katja