Re: RfR JDK-8078335

2015-06-10 Thread Erik Joelsson
Change looks fine, but as David pointed out, please run through jprt. /Erik On 2015-06-11 00:05, Pete Brunet wrote: Please review this simple patch which causes jdk.accessibility to be built for all platforms, not just for Windows. I tested it on Win and Mac. http://cr.openjdk.java.net/~ptbrun

Re: RfR JDK-8078335

2015-06-10 Thread David Holmes
On 11/06/2015 8:05 AM, Pete Brunet wrote: Please review this simple patch which causes jdk.accessibility to be built for all platforms, not just for Windows. I tested it on Win and Mac. http://cr.openjdk.java.net/~ptbrunet/JDK-8078335/webrev.00/ Looks okay but please ensure you test on all pla

RfR JDK-8078335

2015-06-10 Thread Pete Brunet
Please review this simple patch which causes jdk.accessibility to be built for all platforms, not just for Windows. I tested it on Win and Mac. http://cr.openjdk.java.net/~ptbrunet/JDK-8078335/webrev.00/ Pete