Hi Max,
KeytoolChangeAlias.java:
- maybe better to run ks.deleteEntry("13579") in a finally block
- should it delete "246810" as well?
Otherwise looks good to me :)
Artem
2018-03-27 9:51 GMT+02:00 Weijun Wang <weijun.w...@oracle.com>:
> Ping again.
>
> > On Mar 8, 2018, at 8:37 PM, Weijun Wang <weijun.w...@oracle.com> wrote:
> >
> >
> >
> >> 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
> >>
> >
>
>
