Re: RFR: JDK-8157336: Generation of classlists at build time should be configurable

2016-05-24 Thread David Holmes
Hi Erik, On 24/05/2016 11:28 PM, Erik Joelsson wrote: Generating a classlist at build time is not supported on all JVM configurations. This patch adds a configure flag to control this build step: --disable-generate-classlist. The default is to be enabled if either a client or server JVM Variant

Re: RFR: JDK-8157336: Generation of classlists at build time should be configurable

2016-05-24 Thread Tim Bell
Erik: Generating a classlist at build time is not supported on all JVM configurations. This patch adds a configure flag to control this build step: --disable-generate-classlist. The default is to be enabled if either a client or server JVM Variant is being built. Bug: https://bugs.openjdk.ja

RFR: JDK-8157336: Generation of classlists at build time should be configurable

2016-05-24 Thread Erik Joelsson
Generating a classlist at build time is not supported on all JVM configurations. This patch adds a configure flag to control this build step: --disable-generate-classlist. The default is to be enabled if either a client or server JVM Variant is being built. Bug: https://bugs.openjdk.java.net/b

Re: RFR: 8157605 Can't set both CONCURRENCY and EXTRA_JTREG_OPTIONS when running tests

2016-05-24 Thread Staffan Larsen
Thanks! > On 24 maj 2016, at 11:58, Erik Joelsson wrote: > > Looks good. > > /Erik > > On 2016-05-23 20:14, Staffan Larsen wrote: >> This does not work: >> >> $ cd test >> $ make jtreg_tests CONCURRENCY=10 EXTRA_JTREG_OPTIONS=-k:\!headful >> >> I can specify either CONCURRENCY or EXTRA_JTREG

Re: RFR: 8157605 Can't set both CONCURRENCY and EXTRA_JTREG_OPTIONS when running tests

2016-05-24 Thread Erik Joelsson
Looks good. /Erik On 2016-05-23 20:14, Staffan Larsen wrote: This does not work: $ cd test $ make jtreg_tests CONCURRENCY=10 EXTRA_JTREG_OPTIONS=-k:\!headful I can specify either CONCURRENCY or EXTRA_JTREG_OPTIONS, but not both. Please review the fix below: bug: https://bugs.openjdk.java.ne