<div dir="ltr">Hi Max,<div><br></div><div>KeytoolChangeAlias.java:</div><div>- maybe better to run ks.deleteEntry("13579") in a finally block<br></div><div>- should it delete "246810" as well?<br></div><div><br></div><div>Otherwise looks good to me :)</div><div><br></div><div>Artem</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-27 9:51 GMT+02:00 Weijun Wang <span dir="ltr"><<a href="mailto:weijun.wang@oracle.com" target="_blank">weijun.wang@oracle.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping again.<br>
<div class="HOEnZb"><div class="h5"><br>
> On Mar 8, 2018, at 8:37 PM, Weijun Wang <<a href="mailto:weijun.wang@oracle.com">weijun.wang@oracle.com</a>> wrote:<br>
><br>
><br>
><br>
>> On Mar 8, 2018, at 6:13 PM, Weijun Wang <<a href="mailto:weijun.wang@oracle.com">weijun.wang@oracle.com</a>> wrote:<br>
>><br>
>> Please take a review at<br>
>><br>
>>  <a href="http://cr.openjdk.java.net/~weijun/8180570/webrev.00" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~<wbr>weijun/8180570/webrev.00</a><br>
><br>
> Updated in place.<br>
><br>
>><br>
>> Several notes:<br>
>><br>
>> 1. The original KeyStoreCompatibilityMode.sh does not check the exit value of each command. I assume it should.<br>
>><br>
>> 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.<br>
>><br>
>> 3. SecurityTools::<wbr>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/<wbr>./urandom causes an error.<br>
>><br>
>> 4. nonUniqueAliases/<wbr>NonUniqueAliases.sh is not touched. It has an @ignore.<br>
><br>
> This one is also updated. I tried it on a Windows Server 2016 with certutil.exe and it succeeds.<br>
><br>
> --Max<br>
><br>
>><br>
>> Thanks<br>
>> Max<br>
>><br>
><br>
<br>
</div></div></blockquote></div><br></div>