Integrated: 8317807: JAVA_FLAGS removed from jtreg running in JDK-8317039

Magnus Ihse Bursie ihse at openjdk.org
Mon Oct 23 11:29:43 UTC 2023


On Fri, 20 Oct 2023 14:25:30 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> Unfortunately, [JDK-8317039](https://bugs.openjdk.org/browse/JDK-8317039) caused a regression. The `$(JAVA)` variable did not just contain a path, which was assumed by the patch, but also the necessary flags in `$(JAVA_FLAGS)`.
> 
> Furthermore, the original implementation deviated from how we typically treat JDKs in the build system, that is, we store the root to the JDK as the primary setting, and then we use that as base to get a fixpath'ed executable command line. I've changed the code to more closely align to our existing uses of external JDKs.

This pull request has now been integrated.

Changeset: 99de9bb8
Author:    Magnus Ihse Bursie <ihse at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/99de9bb83ff70fe81c89751516a86a94c8f552be
Stats:     14 lines in 4 files changed: 5 ins; 3 del; 6 mod

8317807: JAVA_FLAGS removed from jtreg running in JDK-8317039

Reviewed-by: erikj

-------------

PR: https://git.openjdk.org/jdk/pull/16288


More information about the build-dev mailing list