Code review request: 7102106: TEST_BUG: sun/security/util/Oid/S11N.sh should be modified

[Sean Mullan]

On 7/16/12 9:36 PM, Weijun Wang wrote:
> Also, the test skips an old JDK (and shows a warning) if its path does
> not exist. What this bug wants to solve is that the path exists but is
> too slow.

Ok, got it now.

--Sean

> 
> -Max
> 
> 
> On 07/16/2012 10:43 PM, Wei-jun Wang wrote:
>> The interop test is bi-directional. The new JDK needs to understand what older JDKs produce, vice versa. Therefore it must launch those older JDKs.
>>
>> Thanks
>> Max
>>
>>
>>
>> 在 Jul 16, 2012，10:22 PM，Sean Mullan <sean.mullan at oracle.com> 写道：
>>
>>> Hi Max,
>>>
>>> The fix looks fine, but I am wondering if it is appropriate to require that everyone that runs these tests must have these older JREs installed or mounted for this test to pass. It seems like it would have been better to pre-generate the serialized data and add them as test data files. I know binary files are discouraged, but in this case it should be relatively easy to reproduce the serialized forms if we ever need to, and other tests already do this - for example see test/java/security/BasicPermission/SerialVersion.java
>>>
>>> --Sean
>>>
>>> On 07/16/2012 09:18 AM, Weijun Wang wrote:
>>>> Hi
>>>>
>>>> Please take a review at
>>>>
>>>>    http://cr.openjdk.java.net/~weijun/7102106/webrev.00/
>>>>
>>>> The new environment variable allows you to set an alternative path to
>>>> old JDK releases if the default /java/re/j2se is not what you want.
>>>>
>>>> Thanks
>>>> Max
>>>>
>>>>
>>>> -------- Original Message --------
>>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7102106
>>>> 7102106: TEST_BUG: sun/security/util/Oid/S11N.sh should be modified
>>>>
>>>> === *Description*
>>>> ============================================================
>>>> test used "/java/re" for latest binaries and failed because it is slowly
>>>> or not available from some russia lab machines (timeout)
>>>>
>>>> test show wrong status "Passed" if /java/re is not available on machine
>>>> (reproducible on stt-52.ru.oracle.com)
>>>>
>>>> latest binaries can be defined by env variable(s) and than lines:
>>>>
>>>> OLDJAVA="
>>>>      /java/re/j2se/1.6.0/latest/binaries/${PF}
>>>>      /java/re/j2se/1.5.0/latest/binaries/${PF}
>>>>      /java/re/j2se/1.4.2/latest/binaries/${PF}
>>>> "
>>>>
>>>> can be replaced, for example, on following lines:
>>>> OLDJAVA="
>>>>      ${JDK_LATEST_BASE_PATH}${FS}1.6.0${FS}latest${FS}binaries${FS}${PF}
>>>>      ${JDK_LATEST_BASE_PATH}${FS}1.5.0${FS}latest${FS}binaries${FS}${PF}
>>>>      ${JDK_LATEST_BASE_PATH}${FS}1.4.2${FS}latest${FS}binaries${FS}${PF}
>>>> "
>>>>
>>>> JDK_LATEST_BASE_PATH can be set to /java/re by default and redefined to
>>>> available location if needed
>>>>
>>>> test passed after relocate latest binaries at local Russia server and
>>>> change OLDJAVA
>>>
>>>
>