On 5/10/2016 4:23 PM, Xuelei Fan wrote:
> On 5/10/2016 2:37 PM, Wang Weijun wrote:
>>
>>> On May 10, 2016, at 7:58 AM, Xuelei Fan <[email protected]> wrote:
>>>
>>> On 5/10/2016 12:16 AM, Michael StJohns wrote:
>>>> You're going to hate me, but shouldn't this be checking the Provider
>>>> rather than the OS when trying to determine which test to ignore?
>>>>
>>> It's more accurate to check the provider, too.
>>>
>>> - if (!System.getProperty("os.name").equals("SunOS")) {
>>> + if (provider.getName().equals("SunPKCS11-Solaris") &&
>>> + !System.getProperty("os.name").equals("SunOS")) {
>>
>> There is SunPKCS11-Solaris not on SunOS?
>>
> SunPKPCS11-NSS may be on SunOS.
>
Update the block to:
- if (provider.getName().equals("SunPKCS11-Solaris") &&
- !System.getProperty("os.name").equals("SunOS")) {
+ if (!provider.getName().equals("SunPKCS11-Solaris")) {
This update will be integrated with JDK-8156716. As this is a simple
fix, will not open another thread any more.
Thanks,
Xuelei
