On 2/09/2016 6:46 PM, David Holmes wrote:
On 2/09/2016 5:47 PM, Sharath Ballal wrote:
Hi David,
I had a similar doubt and checked with Christian Tornqvist regarding
the same.  He mentioned that the directory containing the new test is
already part of two test groups, hence I need not add anything for the
new test.

The sun/tools folder is part of two test groups:

jdk_launcher = \
    tools/launcher \
    sun/tools

svc_tools = \
    com/sun/tools/attach \
    sun/tools \
    -sun/tools/java \
    -sun/tools/jrunscript \
    sun/jvmstat \
    demo/jvmti

The test needs to remain in the needs_compact3 group, neither of the
above groups contribute to that.

Sorry - by moving to where you have the test is now in the needs_jdk group so that is fine.

Thanks,
david

David
-----


-Sharath Ballal



-----Original Message-----
From: David Holmes
Sent: Friday, September 02, 2016 1:07 PM
To: Sharath Ballal; serviceability-dev@openjdk.java.net
Subject: Re: RFR: JDK-8165114: stale reference to hotspot test
Test8028623.java

Hi Sharath,

On 1/09/2016 6:00 PM, Sharath Ballal wrote:
Hello,

Please review this fix to remove stale test reference:

Issue: https://bugs.openjdk.java.net/browse/JDK-8165114

Webrev: http://cr.openjdk.java.net/~sballal/8165114/webrev.00/

If the test has moved then the entry has to move from
hotspot/test/TEST.groups to jdk/test/TEST.groups.

Thanks,
David



-Sharath Ballal





Reply via email to