On 2014-07-15 20:04, Mike Duigou wrote:
This looks like a nice improvement and provides a good way to execute specific 
sub-sets that are smaller than the TEST.groups definitions.

I'd like to hook it up to the top level make as an alternative to the current 
recipe.

make test TEST="jdk_core"

Perhaps adjust the top level make test target so that it invokes the test 
makefile with both the contents of TEST and TESTDIRS as two separate executions?

This sounds like a very reasonable follow-up!

Thanks for the review.

Cheers,
Mikael


Mike

On Jul 15 2014, at 19:51 , Mikael Vidstedt <mikael.vidst...@oracle.com> wrote:

I suppose a webrev helps:

http://cr.openjdk.java.net/~mikael/webrevs/8050825/webrev.00/webrev/

Sorry 'bout that.

Cheers,
Mikael

On 2014-07-15 19:48, Mikael Vidstedt wrote:
Please review the below change which adds support for running jtreg tests from the 
top level test/ directory using the 'make TESTDIRS=<path> jtreg_tests' syntax. 
The TESTDIRS syntax is already used in files like hotspot/test/Makefile and 
jdk/test/Makefile and allows for selecting which jtreg tests to run by providing a 
directory/path filter. The change enables doing the same type of invocation from the 
top level; something like this:

cd test && make TESTDIRS=../hotspot/test/runtime jtreg_tests
cd test && make TESTDIRS=../jdk/test/javax jtreg_tests

The implementation logic simply extracts the component (hotspot, jdk etc.) from the 
value of TESTDIRS and delegates to the respective component's test/Makefile, removing 
the ../<component>/test from TESTDIRS in the process.

Thanks,
Mikael


Reply via email to