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

Sean Mullan sean.mullan at oracle.com
Tue Jul 17 19:36:24 UTC 2012

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/viewbug.do?bugid=7102106 7102106: TESTBUG: 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=" ${JDKLATESTBASEPATH}${FS}1.6.0${FS}latest${FS}binaries${FS}${PF} ${JDKLATESTBASEPATH}${FS}1.5.0${FS}latest${FS}binaries${FS}${PF} ${JDKLATESTBASEPATH}${FS}1.4.2${FS}latest${FS}binaries${FS}${PF} " JDKLATESTBASEPATH 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

More information about the security-dev mailing list