Looks good.

Thanks,
David

On 21/10/2014 9:58 PM, Erik Joelsson wrote:
Please review this fix for a simple typo, causing the top level test
target to not work.

Bug: https://bugs.openjdk.java.net/browse/JDK-8061610
Patch inline:
--- old/make/Main.gmk    2014-10-21 13:44:35.237130786 +0200
+++ new/make/Main.gmk    2014-10-21 13:44:35.137134378 +0200
@@ -286,7 +286,7 @@

  test:
      ($(CD) $(SRC_ROOT)/test && $(MAKE) $(MAKE_ARGS) -j1 -k MAKEFLAGS= \
-        JT_HOME=$(JT_HOME) PRODUCT_HOME=$(JDK_OUTPUT_DIR) \
+        JT_HOME=$(JT_HOME) PRODUCT_HOME=$(JDK_OUTPUTDIR) \
          ALT_OUTPUTDIR=$(OUTPUT_ROOT) CONCURRENCY=$(JOBS) $(TEST)) || true

  test-make:


/Erik

Reply via email to