It looks good.
A question:
Do we want to get rid of the jhat in the jdk/closed/old-build/common/Release.gmk ?
    jhat.1             \
    com/sun/tools/hat       \
    jhat$(EXE_SUFFIX)

Thanks,
Serguei



On 4/23/15 5:06 AM, Staffan Larsen wrote:
Please review this change to remove the jhat tool. I will not push this change until the tests have been removed (see a different review thread).

JEP: https://bugs.openjdk.java.net/browse/JDK-8059039
bug: https://bugs.openjdk.java.net/browse/JDK-8076473

webrev root: http://cr.openjdk.java.net/~sla/8076473/webrev.root.00/ <http://cr.openjdk.java.net/%7Esla/8076473/webrev.root.00/> webrev jdk: http://cr.openjdk.java.net/~sla/8076473/webrev.jdk.00/ <http://cr.openjdk.java.net/%7Esla/8076473/webrev.jdk.00/>

Thanks,
/Staffan


Reply via email to