> On Mar 8, 2018, at 6:13 PM, Weijun Wang <weijun.w...@oracle.com> wrote:
> 
> Please take a review at
> 
>   http://cr.openjdk.java.net/~weijun/8180570/webrev.00

Updated in place.

> 
> Several notes:
> 
> 1. The original KeyStoreCompatibilityMode.sh does not check the exit value of 
> each command. I assume it should.
> 
> 2. I don't really understand what SystemDrive is for in RSAEncryptDecrypt.sh. 
> Maybe about running the test from \\remote\share? I also cannot find 
> JDK-6449799.
> 
> 3. SecurityTools::getProcessBuilder is modified a little. On my Windows 
> machine without cygwin (major reason why I want to work on this RFE), setting 
> -Djava.security.egd=file:/dev/./urandom causes an error.
> 
> 4. nonUniqueAliases/NonUniqueAliases.sh is not touched. It has an @ignore.

This one is also updated. I tried it on a Windows Server 2016 with certutil.exe 
and it succeeds.

--Max

> 
> Thanks
> Max
> 

Reply via email to