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
