Review request: JDK-8006753 fix failed for JDK-8002415 White box testing API for HotSpot

[Christian Törnqvist]

Looks good!

Thanks,
Christian

-----Original Message-----
From: Mikael Gerdin 
Sent: den 4 februari 2013 15:00
To: hotspot-dev at openjdk.java.net
Subject: Review request: JDK-8006753 fix failed for JDK-8002415 White box testing API for HotSpot

Hi,

Background:
Even from the beginning of the WhiteBox testing API implementation the tests have not worked on builds promoted by release engineering due to the fact that the wb.jar file containing the Java classes was not imported by the JDK-level makefiles.
On one level this was ok since we didn't want to ship the classes to end users and we worked around the complexity of somehow post-processing the RE bundles to remove them.

The problem is that we aren't able to run the WhiteBox tests in promotion testing, or to verify fixes against earlier builds.

Solution:
The solution suggested here is to put the WhiteBox helper classes in the testlibrary directory under test/ and
* tell jtreg to first compile the classes
* run a special Java program to copy the classes to the test work directory
* run the white box test with -Xbootclasspath/a:. to pick up the copied classes.

The solution is kind-of hackish but I've been unable to come up with a better solution without performing complex surgery on jtreg.

The changes to the make/ directory should basically revert the makefile changes that were added to support this API.

Webrev:
http://cr.openjdk.java.net/~mgerdin/8006753/webrev.0/
Bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8006753

Note about renamed/moved files:
I will try to make sure that the moved/renamed files will be tracked properly by mercurial, but I think that using patch files and MQ breaks this so I'll do this after the review is done.

Thanks
/Mikael