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

[Igor Ignatyev]

Hi Mikael,

I think this will make more uncomfortable to use WhiteBox with harnesses 
other than jtreg. e.g. for tonga:
now we can just put wb.jar to classpath and use it. but after your 
changes we will need to build it ourselves before running tests.

Best regards,
Igor Ignatyev

On 02/04/2013 05:59 PM, Mikael Gerdin wrote:
> 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