RFR 8046002: Move Ucrypto to the open jdk repo

Sean Mullan sean.mullan at oracle.com
Wed Oct 15 18:58:00 UTC 2014


Looks good.

--Sean

On 10/15/2014 12:56 PM, Valerie Peng wrote:
>
> Sure, webrev updated at
> http://cr.openjdk.java.net/~valeriep/8046002/webrev.01.
> Thanks for the prompt review,
> Valerie
>
> On 10/14/2014 5:18 AM, Sean Mullan wrote:
>> You should change the @since tags to 1.9 since this is its first
>> inclusion in OpenJDK.
>>
>> Looks good otherwise.
>>
>> --Sean
>>
>> On 10/13/2014 07:54 PM, Valerie Peng wrote:
>>> Hi, Sean,
>>>
>>> Could you please review the changes for 8046002: Move Ucrypto to the
>>> open jdk repo?
>>> Once you reviewed it, I will request a review from the build team since
>>> there are build-related changes.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8046002
>>> Webrev: http://cr.openjdk.java.net/~valeriep/8046002/webrev.00/
>>


More information about the security-dev mailing list