Code review request: 8017453: ReplayCache tests fail on multiple platforms (was Re: Please be prepared for possible test failures (was Fwd: [JBS] (JDK-8001326) Improve Kerberos replay caching))

[Weijun Wang]

Oh, actually it should be SunOS.

On 6/24/13 11:28 AM, Xuelei Fan wrote:
> On 6/24/2013 11:04 AM, Weijun Wang wrote:
>> Oh, no
>>
>>   if (!os.startsWith("Solaris") && !os.startsWith("Linux")) {
>>
> Good.
> 
> Xuelei
> 
>> --Max
>>
>> On 6/24/13 11:01 AM, Weijun Wang wrote:
>>> How about
>>>
>>>               if (!os.startsWith("Solaris") || !os.startsWith("Linux")) {
>>>                   mode = -1;
>>>               }
>>>
>>> Thanks
>>> Max
>>>
>>>
>>> On 6/24/13 10:35 AM, Xuelei Fan wrote:
>>>> On 6/24/2013 10:01 AM, Weijun Wang wrote:
>>>>>
>>>>>
>>>>> On 6/24/13 9:49 AM, Xuelei Fan wrote:
>>>>>> ReplayCacheTestProc.java
>>>>>> ========================
>>>>>> 75       String os = System.getProperty("os.name", "???");
>>>>>>
>>>>>> What happens if "os" is "???"?
>>>>>
>>>>> Then it's neither Windows nor Mac OS X. :)
>>>>>
>>>> The code means except Windows and Mac OS X, all other platform supports
>>>> native mode, even the OS name is unknown.
>>>>
>>>>> I gave a default value so that the result will not be null. Although the
>>>>> system property "os.name" should always have a value, it makes me
>>>>> comfort not thinking of possible NPE.
>>>>>
>>>> ;-) I guess you may also worry that if the OS does not support native
>>>> mode other than Windows and Mac OS X, the test will fail again.
>>>>
>>>> As you are here already, I would suggest to make more improvement. When
>>>> the OS is unknown, java mode should be used .
>>>>
>>>> Xuelei
>>>>
>>>>> Thanks
>>>>> Max
>>>>>
>>>>>>
>>>>>> Xuelei
>>>>>>
>>>>>> On 6/24/2013 12:36 AM, Wang Weijun wrote:
>>>>>>> Send again. And BTW, JPRT runs fine.
>>>>>>>
>>>>>>> 在 Jun 23, 2013，6:29 PM，Weijun Wang <weijun.wang at oracle.com> 写道：
>>>>>>>
>>>>>>>> The macosx problem found, the machine's native GSS does not support shared replaycache.
>>>>>>>>
>>>>>>>> *Valerie* and/or *Xuelei*, can you please review the fix?
>>>>>>>>
>>>>>>>>       http://cr.openjdk.java.net/~weijun/8017453/webrev.00/
>>>>>>>>
>>>>>>>>
>