Hi Staffan,

Changes look good.

However it looks like this test has the same issue, right?
http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/file/59d3a69564dc/test/gc/g1/plab/TestPLABResize.java

It is currently ignored, but we should probably fix it too...

Thanks,
Bengt

On 2016-03-09 10:47, Staffan Larsen wrote:
Please review this small fix to some tests with the wrong paths in them. Tested 
locally with jtreg 4.1 b12 and b13.

Thanks,
/Staffan


$ hg diff
diff --git a/test/gc/g1/plab/TestPLABPromotion.java 
b/test/gc/g1/plab/TestPLABPromotion.java
--- a/test/gc/g1/plab/TestPLABPromotion.java
+++ b/test/gc/g1/plab/TestPLABPromotion.java
@@ -27,7 +27,7 @@
   * @summary Test PLAB promotion
   * @requires vm.gc=="G1" | vm.gc=="null"
   * @requires vm.opt.FlightRecorder != true
- * @library /testlibrary /../../test/lib /
+ * @library /testlibrary /test/lib /
   * @modules java.management
   * @build ClassFileInstaller
   *        sun.hotspot.WhiteBox
diff --git a/test/serviceability/dcmd/gc/HeapDumpAllTest.java 
b/test/serviceability/dcmd/gc/HeapDumpAllTest.java
--- a/test/serviceability/dcmd/gc/HeapDumpAllTest.java
+++ b/test/serviceability/dcmd/gc/HeapDumpAllTest.java
@@ -35,7 +35,7 @@
   * @build jdk.test.lib.hprof.*
   * @build jdk.test.lib.hprof.model.*
   * @build jdk.test.lib.hprof.parser.*
- * @build jdk.test.lib.hprof.utils.*
+ * @build jdk.test.lib.hprof.util.*
   * @build HeapDumpTest
   * @run testng HeapDumpAllTest
   */
diff --git a/test/serviceability/dcmd/gc/HeapDumpTest.java 
b/test/serviceability/dcmd/gc/HeapDumpTest.java
--- a/test/serviceability/dcmd/gc/HeapDumpTest.java
+++ b/test/serviceability/dcmd/gc/HeapDumpTest.java
@@ -51,7 +51,7 @@
   * @build jdk.test.lib.hprof.*
   * @build jdk.test.lib.hprof.model.*
   * @build jdk.test.lib.hprof.parser.*
- * @build jdk.test.lib.hprof.utils.*
+ * @build jdk.test.lib.hprof.util.*
   * @run testng HeapDumpTest
   */
  public class HeapDumpTest {

Reply via email to