Updated webrev looks good.
Thanks,
Valerie
On 4/30/2019 6:24 PM, sha.ji...@oracle.com wrote:
Hi Valeria,
On 2019/5/1 04:25, Valerie Peng wrote:
Looks fine to me.
Thanks for your review!
With further testing, sun/security/tools/keytool/NssTest.java may fail
due to some dependencies are not found.
But this failure should not be related to this NSS libs upgrade,
because it could also raise without this fix.
So, just loading the dependencies explicitly.
Please review the updated webrev:
http://cr.openjdk.java.net/~jjiang/8204203/webrev.01/
Just curious, how do we see the changes on the artifactory side?
Please search the below files in our Artifactory.
nsslib-macosx_x64-3.41.zip
nsslib-windows_x64-3.41-VS2017.zip
nsslib-windows_x86-3.41-VS2017.zip
Best regards,
John Jiang
Valerie
On 4/23/2019 7:39 PM, sha.ji...@oracle.com wrote:
Hi,
NSS 3.41 has been approved, so just build this version for windows
(with VS2017) and macosx.
Webrev: http://cr.openjdk.java.net/~jjiang/8204203/webrev.00/
Issue: https://bugs.openjdk.java.net/browse/JDK-8204203
Best regards,
John Jiang
