RFR: JDK-8159186: jdk/test/Makefile: allow users to set verbosity
Dan Smith
daniel.smith at oracle.com
Thu Jun 9 18:17:49 UTC 2016
I'd like to make a simple tweak to jdk/test/Makefile in order to support control over the "-v" verbosity option sent to jtreg. 'langtools/test/Makefile' has supported a similar "JTREG_VERBOSE" variable for awhile now.
(Would it be nice to align the test-running infrastructure for langtools and jdk more closely, and maybe integrate better into the rest of the Makefile setup? Sure, yes. Totally out of scope for what I'm doing here, though, which is just trying to eliminate a small pain point.)
# HG changeset patch
# Parent b803887ddddc681a9cfa067a1a4ccab70c465a53
diff -r b803887ddddc -r b629baacaa5b test/Makefile
--- a/test/Makefile Thu Jun 02 15:36:15 2016 +0300
+++ b/test/Makefile Wed Jun 08 15:20:04 2016 -0600
@@ -316,8 +316,9 @@
# Always turn on assertions
JTREG_ASSERT_OPTION = -ea -esa
JTREG_BASIC_OPTIONS += $(JTREG_ASSERT_OPTION)
-# Report details on all failed or error tests, times too
-JTREG_BASIC_OPTIONS += -v:fail,error,time
+# jtreg verbosity setting
+JTREG_VERBOSE ?= fail,error,time
+JTREG_BASIC_OPTIONS += $(if $(JTREG_VERBOSE),-v:$(JTREG_VERBOSE))
# Retain all files for failing tests
JTREG_BASIC_OPTIONS += -retain:fail,error
# Ignore tests are not run and completely silent about it
Can I get a reviewer's approval?
—Dan
More information about the build-dev
mailing list