The CSR looks fine to me. Thanks.
-coderaptor On Wed, Nov 7, 2018 at 7:53 AM Weijun Wang <weijun.w...@oracle.com> wrote: > > Oops, I take this back. The CSR needs more update. > > Sorry if you have already start reading it. > > Thanks > Max > > > > On Nov 7, 2018, at 9:27 AM, Weijun Wang <weijun.w...@oracle.com> wrote: > > > > After some discussion, we decided to cover -keysize and -sigalg in this > > deprecation process too. > > > > Please review the updated CSR at > > > > https://bugs.openjdk.java.net/browse/JDK-8212111 > > > > No webrev available yet. > > > > Thanks > > Max > > > > > >> On Oct 18, 2018, at 10:34 AM, Weijun Wang <weijun.w...@oracle.com> wrote: > >> > >> Please review the code change and CSR for > >> > >> JBS: https://bugs.openjdk.java.net/browse/JDK-8212003 > >> > >> at > >> > >> webrev: http://cr.openjdk.java.net/~weijun/8212003/webrev.00/ > >> CSR: https://bugs.openjdk.java.net/browse/JDK-8212111 > >> > >> When -keyalg is not provided for -genkeypair or -genseckey, keytool will > >> print out a warning. We plan to make this an error in a future release. > >> > >> A new regression test ObsoleteKeyalg.java added. "-keyalg DSA" or "-keyalg > >> DES" added to other tests. > >> > >> A Mach5 job on tier1 and tier2 running now. > >> > >> Thanks > >> Max > >> > > >
