Skip to content

Commit f27e8c8

Browse files
committed
8317039: Enable specifying the JDK used to run jtreg
8317807: JAVA_FLAGS removed from jtreg running in JDK-8317039 Backport-of: b186446511655b317e05c79aeb3cdd324e254cdc
1 parent 3134d7e commit f27e8c8

File tree

5 files changed

+44
-2
lines changed

5 files changed

+44
-2
lines changed

make/RunTests.gmk

+1-1
Original file line numberDiff line numberDiff line change
@@ -749,7 +749,7 @@ define SetupRunJtregTestBody
749749
$$(RM) -r $$($1_TEST_RESULTS_DIR)
750750

751751
$1_COMMAND_LINE := \
752-
$$(JAVA) $$($1_JTREG_LAUNCHER_OPTIONS) \
752+
$$(JTREG_JAVA) $$($1_JTREG_LAUNCHER_OPTIONS) \
753753
-Dprogram=jtreg -jar $$(JT_HOME)/lib/jtreg.jar \
754754
$$($1_JTREG_BASIC_OPTIONS) \
755755
-testjdk:$$(JDK_IMAGE_DIR) \

make/RunTestsPrebuilt.gmk

+2
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,7 @@ $(eval $(call SetupVariable,JT_HOME))
122122
$(eval $(call SetupVariable,JDK_IMAGE_DIR,$(OUTPUTDIR)/images/jdk))
123123
$(eval $(call SetupVariable,TEST_IMAGE_DIR,$(OUTPUTDIR)/images/test))
124124
$(eval $(call SetupVariable,SYMBOLS_IMAGE_DIR,$(OUTPUTDIR)/images/symbols,NO_CHECK))
125+
$(eval $(call SetupVariable,JTREG_JDK,$(BOOT_JDK)))
125126

126127
# Provide default values for tools that we need
127128
$(eval $(call SetupVariable,MAKE,make,NO_CHECK))
@@ -294,6 +295,7 @@ $(call CreateNewSpec, $(NEW_SPEC), \
294295
TOPDIR := $(TOPDIR), \
295296
OUTPUTDIR := $(OUTPUTDIR), \
296297
BOOT_JDK := $(BOOT_JDK), \
298+
JTREG_JDK := $(JTREG_JDK), \
297299
JT_HOME := $(JT_HOME), \
298300
JDK_IMAGE_DIR := $(JDK_IMAGE_DIR), \
299301
TEST_IMAGE_DIR := $(TEST_IMAGE_DIR), \

make/RunTestsPrebuiltSpec.gmk

+2
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,8 @@ JAR := $(FIXPATH) $(JAR_CMD)
125125
JLINK := $(FIXPATH) $(JLINK_CMD)
126126
JMOD := $(FIXPATH) $(JMOD_CMD)
127127

128+
JTREG_JAVA := $(FIXPATH) $(JTREG_JDK)/bin/java $(JAVA_FLAGS_BIG) $(JAVA_FLAGS)
129+
128130
BUILD_JAVA := $(JDK_IMAGE_DIR)/bin/JAVA
129131
################################################################################
130132
# Some common tools. Assume most common name and no path.

make/autoconf/lib-tests.m4

+36-1
Original file line numberDiff line numberDiff line change
@@ -145,12 +145,47 @@ AC_DEFUN_ONCE([LIB_TESTS_SETUP_JTREG],
145145
UTIL_FIXUP_PATH(JT_HOME)
146146
AC_SUBST(JT_HOME)
147147
148+
# Specify a JDK for running jtreg. Defaults to the BOOT_JDK.
149+
AC_ARG_WITH(jtreg-jdk, [AS_HELP_STRING([--with-jdk],
150+
[path to JDK for running jtreg @<:@BOOT_JDK@:>@])])
151+
152+
AC_MSG_CHECKING([for jtreg jdk])
153+
if test "x${with_jtreg_jdk}" != x; then
154+
if test "x${with_jtreg_jdk}" = xno; then
155+
AC_MSG_RESULT([no, jtreg jdk not specified])
156+
elif test "x${with_jtreg_jdk}" = xyes; then
157+
AC_MSG_RESULT([not specified])
158+
AC_MSG_ERROR([--with-jtreg-jdk needs a value])
159+
else
160+
JTREG_JDK="${with_jtreg_jdk}"
161+
AC_MSG_RESULT([$JTREG_JDK])
162+
UTIL_FIXUP_PATH(JTREG_JDK)
163+
if test ! -f "$JTREG_JDK/bin/java"; then
164+
AC_MSG_ERROR([Could not find jtreg java at $JTREG_JDK/bin/java])
165+
fi
166+
fi
167+
else
168+
JTREG_JDK="${BOOT_JDK}"
169+
AC_MSG_RESULT([no, using BOOT_JDK])
170+
fi
171+
172+
UTIL_FIXUP_PATH(JTREG_JDK)
173+
AC_SUBST([JTREG_JDK])
174+
# For use in the configure script
175+
JTREG_JAVA="$FIXPATH $JTREG_JDK/bin/java"
176+
148177
# Verify jtreg version
149178
if test "x$JT_HOME" != x; then
179+
AC_MSG_CHECKING([jtreg jar existence])
180+
if test ! -f "$JT_HOME/lib/jtreg.jar"; then
181+
AC_MSG_ERROR([Could not find jtreg jar at $JT_HOME/lib/jtreg.jar])
182+
fi
183+
150184
AC_MSG_CHECKING([jtreg version number])
151185
# jtreg -version looks like this: "jtreg 6.1+1-19"
152186
# Extract actual version part ("6.1" in this case)
153-
jtreg_version_full=`$JAVA -jar $JT_HOME/lib/jtreg.jar -version | $HEAD -n 1 | $CUT -d ' ' -f 2`
187+
jtreg_version_full=$($JTREG_JAVA -jar $JT_HOME/lib/jtreg.jar -version | $HEAD -n 1 | $CUT -d ' ' -f 2)
188+
154189
jtreg_version=${jtreg_version_full/%+*}
155190
AC_MSG_RESULT([$jtreg_version])
156191

make/autoconf/spec.gmk.in

+3
Original file line numberDiff line numberDiff line change
@@ -652,6 +652,9 @@ SJAVAC_SERVER_JAVA = $(SJAVAC_SERVER_JAVA_CMD) \
652652
# overriding that value by using ?=.
653653
JAVAC_FLAGS? = @JAVAC_FLAGS@
654654

655+
JTREG_JDK := @JTREG_JDK@
656+
JTREG_JAVA = @FIXPATH@ $(JTREG_JDK)/bin/java $(JAVA_FLAGS_BIG) $(JAVA_FLAGS)
657+
655658
BUILD_JAVA_FLAGS := @BOOTCYCLE_JVM_ARGS_BIG@
656659
BUILD_JAVA=@FIXPATH@ $(BUILD_JDK)/bin/java $(BUILD_JAVA_FLAGS)
657660
BUILD_JAR=@FIXPATH@ $(BUILD_JDK)/bin/jar

0 commit comments

Comments
 (0)