RFR(xs): 8171225: [aix] Build gtests on AIX 7.1 with xlC 12

2016-12-14 Thread Thomas Stüfe
Hi all, please review this small change. It fixes the gtest build on AIX and enables it by default. Note that even though this is a fix for AIX, a cast needed to be added to shared test coding. This is because xlC struggles with certain template expansions and I had to help it by providing an exp

Re: Review Request: JDK-8171201 & JDK-8171202: Drop java.compact$N aggregator modules

2016-12-14 Thread Alan Bateman
On 14/12/2016 16:19, Mandy Chung wrote: : Fixed. All looks good to me. -Alan

Re: Review Request: JDK-8171201 & JDK-8171202: Drop java.compact$N aggregator modules

2016-12-14 Thread Mandy Chung
> On Dec 14, 2016, at 12:03 AM, Alan Bateman wrote: > > On 14/12/2016 06:49, Mandy Chung wrote: > >> JDK-8171201: Drop java.compact$N aggregator modules >> JDK-8171202: Rename jdk.crypto.pkcs11 and jdk.pack200 to end with Java >> letters >> >> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8

Re: Review Request: JDK-8171201 & JDK-8171202: Drop java.compact$N aggregator modules

2016-12-14 Thread Erik Joelsson
Build changes look good to me. /Erik On 2016-12-14 07:49, Mandy Chung wrote: JDK-8171201: Drop java.compact$N aggregator modules JDK-8171202: Rename jdk.crypto.pkcs11 and jdk.pack200 to end with Java letters http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171201%2b8171202/webrev.00/ Compact

Re: Review Request: JDK-8171201 & JDK-8171202: Drop java.compact$N aggregator modules

2016-12-14 Thread Alan Bateman
On 14/12/2016 06:49, Mandy Chung wrote: JDK-8171201: Drop java.compact$N aggregator modules JDK-8171202: Rename jdk.crypto.pkcs11 and jdk.pack200 to end with Java letters http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8171201%2b8171202/webrev.00/ The hg moves are showing up as new files in t