On Fri, 20 Oct 2023 14:25:30 GMT, Magnus Ihse Bursie <[email protected]> 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 <[email protected]> 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
