RFR: JDK-8149777: Enable enhanced failure handler for "make test"

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Thu Apr 14 10:58:36 UTC 2016


Ok. Lgtm. 

/Magnus

> 14 apr. 2016 kl. 10:24 skrev Erik Joelsson <erik.joelsson at oracle.com>:
> 
> The failure_handler is a jtreg extension that we build and put in the test image. It's then used when running our product tests. As a stand alone product, it has its own set of tests. The old makefile had a target for running those so I felt I needed to keep that functionality. I agree the make target can be confusing, but I couldn't find a better one.
> 
> /Erik
> 
>> On 2016-04-13 12:48, Magnus Ihse Bursie wrote:
>> Is the test-failure-handle used to test that the failure handler is correct? It seems a bit odd, since most other tests has their main purpose to be run, and the build part is just a necessary thing, but here I presume that the main thing is to build the handler so other tests can use it, and it is not clear what the test itself mean. Otoh, I can't see any other way to express this.
>> 
>> /Magnus
>> 
>>> 12 apr. 2016 kl. 16:00 skrev Erik Joelsson <erik.joelsson at oracle.com>:
>>> 
>>> Please review this change which adds a proper makefile and build sequence for the failure-handler jtreg plugin. It also adds the failure handler to the test image and makes the hotspot and jdk test/Makefile's pick it up when available.
>>> 
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8149777
>>> Webrev: http://cr.openjdk.java.net/~erikj/8149777/webrev.03/
>>> 
>>> /Erik
> 


More information about the hotspot-dev mailing list