Code review request: 6988842 jce/ECC test fails for SunPKCS11 provider using nss library

[Valerie (Yu-Ching) Peng]

Vinnie,

Would it clearer to name the new configuration option 
"minLibraryVersion" instead of "libraryVersionCheck"?
Would it help to include the required minimum value in the exception 
message in SunPKCS11.java?

Also, do you know what's the nss version that's commonly available on 
our Solaris and Linux machines based on your testing?

Has there a doc bug been filed for updating our PKCS11 doc w/ this new 
configuration option as well as the minimum NSS version requirement?

Thanks,
Valerie

On 03/02/12 04:23, Vincent Ryan wrote:
> On 03/ 2/12 12:22 PM, Vincent Ryan wrote:
>> Please review the fix for:
>>      6988842 jce/ECC test fails for SunPKCS11 provider using nss library
>>
>> It is the long overdue refresh of the NSS libraries that we include in
>> the JDK regression test suite. One significant change is that the
>> NSS libraries are now obtained from the OS on Solaris and Linux.
>> On Windows the libraries continue to be supplied.
>>
>> Thanks.
>>
> And the webrev:
>     http://cr.openjdk.java.net/~vinnie/6988842/webrev.00/