Re: RFR: JDK-8074016 Add convenient way of adding custom test targets to hotspot's test makefile

2015-04-27 Thread Staffan Larsen
Looks good! Thanks, /Staffan > On 27 apr 2015, at 12:25, Ingemar Aberg wrote: > > Please review this small fix to hotspot's test makefile > > The file hotspot/test/Makefile provides '-include $(ALT_MAKE)/Makefile' close > to the top (before most variables are set). > Move that line to after t

Re: RFR: JDK-8074016 Add convenient way of adding custom test targets to hotspot's test makefile

2015-04-27 Thread Erik Joelsson
Looks ok to me, but you should probably get a hotspot review to look at it too. /Erik On 2015-04-27 03:25, Ingemar Aberg wrote: Please review this small fix to hotspot's test makefile The file hotspot/test/Makefile provides '-include $(ALT_MAKE)/Makefile' close to the top (before most variab

RFR: JDK-8074016 Add convenient way of adding custom test targets to hotspot's test makefile

2015-04-27 Thread Ingemar Aberg
Please review this small fix to hotspot's test makefile The file hotspot/test/Makefile provides '-include $(ALT_MAKE)/Makefile' close to the top (before most variables are set). Move that line to after the variables are defined so they are available to the custom makefile. Bug: https://bugs.o