Looks good to me.
David
On 7/06/2012 4:20 PM, Staffan Larsen wrote:
Here is an updated webrev with an empty jdk_jfr target in the OPENJDK case:
http://cr.openjdk.java.net/~sla/7165257/webrev.02/
Thanks,
/Staffan
On 6 jun 2012, at 00:44, David Holmes wrote:
Hi Staffan,
On 25/05/2012 11:41 PM, Staffan Larsen wrote:
Please review the following change to makefiles to add a target for the
JFR tests.
Webrev: http://cr.openjdk.java.net/~sla/7165257/webrev.01/
As discussed there has to be an empty jfr_test target in the OPENJDK
case, because the JPRT file will cause it to be invoked.
Do you have an updated webrev with the modified fix?
David
-----