RFR: 8317807: JAVA_FLAGS removed from jtreg running in JDK-8317039
Magnus Ihse Bursie
ihse at openjdk.org
Fri Oct 20 14:32:48 UTC 2023
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.
-------------
Commit messages:
- 8317807: JAVA_FLAGS removed from jtreg running in JDK-8317039
Changes: https://git.openjdk.org/jdk/pull/16288/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=16288&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8317807
Stats: 14 lines in 4 files changed: 5 ins; 3 del; 6 mod
Patch: https://git.openjdk.org/jdk/pull/16288.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16288/head:pull/16288
PR: https://git.openjdk.org/jdk/pull/16288
More information about the build-dev
mailing list