> On Nov 16, 2018, at 11:48 PM, Sean Mullan <sean.mul...@oracle.com> wrote:
>
> On 11/15/18 10:20 PM, Weijun Wang wrote:
>> Would like also please review the release note here?
>> https://bugs.openjdk.java.net/browse/JDK-8213965
>
> I made a few changes. I think it is important to also say that we will be
> removing support for the default values in a subsequent release.
Thanks.
>
>> I had thought about using RN-Deprecated but there is no API here. If you
>> think it's better, I'll use it and also change all "obsolete" into
>> "deprecate" in the description and title of the release note/CSR/bug.
>
> I think "deprecate" is probably better, even though this isn't an API,
> because I think it is a better way to tell the user that it is not good to
> rely on the default values anymore. "Obsoleted" is probably ok too, but I
> don't hear it used much so it doesn't have the same meaning to me.
I s/obsolete/deprecate/ everywhere. The webrev is also updated in place.
--Max
>
> --Sean
>
>> Thanks
>> Max
>>> On Nov 15, 2018, at 9:19 AM, Weijun Wang <weijun.w...@oracle.com> wrote:
>>>
>>> Thanks to Xuelei and Sean. I added your recommended words and proposed the
>>> CSR.
>>>
>>>> On Nov 15, 2018, at 6:16 AM, Sean Mullan <sean.mul...@oracle.com> wrote:
>>>>
>>>> On 11/14/18 5:07 AM, Weijun Wang wrote:
>>>>> The CSR is re-opened. It is now focusing on -keyalg only. Please take a
>>>>> review:
>>>>> https://bugs.openjdk.java.net/browse/JDK-8212111
>>>>
>>>> I think the CSR should also include an example of the informational text
>>>> showing what algs and size were used. Looks good otherwise.
>>>>
>>>> --Sean
>>>>
>>>>> Thanks
>>>>> Max
>>>>>> On Nov 7, 2018, at 11:51 PM, 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
>>>>>>>>
>>>>>>>
>>>>>>
>>>