The patch looks okay to me.
Mandy

On 11/12/2013 1:35 PM, Alan Bateman wrote:

This test was liberated earlier today via JDK- 6543856 [1]. Unfortunately it is failing again, this time with a NoClassDefFoundError because it can't locate the JavaProcess class. The issue can be quickly fixed with the attached patched (to add testlibrary to the classpah). Alternatively we can exclude this test again, pending a re-visit of this test and perhaps eliminating the shell script.

-Alan.


diff --git a/test/sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh b/test/sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh
--- a/test/sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh
+++ b/test/sun/jvmstat/monitor/MonitoredVm/MonitorVmStartTerminate.sh
@@ -38,6 +38,6 @@
 verify_os

 JAVA="${TESTJAVA}/bin/java"
-CP=${TESTJAVA}${FS}lib${FS}tools.jar${PS}${TESTCLASSES}
+CP=${TESTJAVA}${FS}lib${FS}tools.jar${PS}${TESTCLASSES}${PS}${TESTCLASSES}${FS}..${FS}..${FS}testlibrary

 ${JAVA} ${TESTVMOPTS} -classpath ${CP} MonitorVmStartTerminate



[1] http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4cff9f59644f

Reply via email to