I also found this test that uses jhat and needs an update: com/sun/management/HotSpotDiagnosticMXBean/DumpHeap.sh
/Staffan > On 22 apr 2015, at 15:05, Staffan Larsen <staffan.lar...@oracle.com> wrote: > > I think you are missing a "@build jdk.test.lib.hprof..*”. > > /Staffan > >> On 22 apr 2015, at 14:25, Yekaterina Kantserova >> <yekaterina.kantser...@oracle.com> wrote: >> >> Staffan, thank you for the review! >> >> This issue needs a change in jdk as well. >> http://cr.openjdk.java.net/~ykantser/8059047.jdk/webrev.00 - >> BasicJMapTest.java will use HprofParser to verify hprof dumps created by the >> test. >> >> // Katja >> >> >> >> On 04/22/2015 01:20 PM, Staffan Larsen wrote: >>> Looks good! >>> >>> Thanks, >>> /Staffan >>> >>>> On 22 apr 2015, at 11:17, Yekaterina Kantserova >>>> <yekaterina.kantser...@oracle.com> wrote: >>>> >>>> Hi, >>>> >>>> Could I please have a review of this fix. >>>> >>>> bug: https://bugs.openjdk.java.net/browse/JDK-8059047 >>>> webrev: http://cr.openjdk.java.net/~ykantser/8059047/webrev.00/ >>>> >>>> This fix is a part of JEP 241: Remove the jhat Tool >>>> (https://bugs.openjdk.java.net/browse/JDK-8059039). I suggest to put >>>> parser/validator into common test library since the functionality can be >>>> useful not only for SVC tools tests but even for some future GC tests. >>>> >>>> The old jhat packages have been moved as follows: >>>> com.sun.tools.hat.internal.model -> jdk.test.lib.hprof.model >>>> com.sun.tools.hat.internal.parser -> jdk.test.lib.hprof.parser >>>> com.sun.tools.hat.internal.util -> jdk.test.lib.hprof.util >>>> >>>> The source has not been changed except Copyrights year. >>>> >>>> Thanks, >>>> Katja >> >
