RFR (S): 7165257: Adding JFR test targets

Kelly O'Hair kelly.ohair at oracle.com
Thu Jun 7 11:45:28 PDT 2012


Looks fine.

-kto

On Jun 6, 2012, at 11: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
>> -----
>> 
>> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20120607/fcc3b070/attachment-0001.html 


More information about the serviceability-dev mailing list