RFR: JDK-8187642: The consolidated repo test makefile disables CONCURRENCY setting to jtreg

[David Holmes]

Hi Erik,

On 19/09/2017 10:53 AM, Erik Joelsson wrote:
> When invoking the test/Makefile using the jtreg_tests target in the 
> consolidated repo, the CONCURRENCY variable gets overridden with an 
> empty value. This was introduced with my changes to handle the new test 
> roots. I believe the proper fix is to surround the override of 
> CONCURRENCY with a conditional.

Why isn't this handled by the preceding block:

   51   ifeq ($(shell $(EXPR) $(JOBS) \> 50), 1)
   52     # JTReg cannot handle more than 50 in concurrency
   53     JDK_TEST_JOBS=50
   54   else
   55     JDK_TEST_JOBS=$(JOBS)
   56   endif
   57 else
   58   JDK_TEST_JOBS=$(TEST_JOBS)
   59 endif

David

> Bug: https://bugs.openjdk.java.net/browse/JDK-8187642
> 
> Webrev: http://cr.openjdk.java.net/~erikj/8187642/webrev.01/
> 
> /Erik
>