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
-----



Reply via email to