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