RFR: 8310183: Update GitHub Actions to use boot JDK for building jtreg

Christian Stein cstein at openjdk.org
Fri Jun 16 09:52:15 UTC 2023


On Fri, 16 Jun 2023 08:45:01 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Wait, so why does it fix the bug? Is it a MSYS path conversion bug?

It does not fix the bug, it works around it. Something (in MSYS) failes to work (calling `javac`) that used to work without problem until some week ago (new version of MSYS?), and looking into the workflow step log, it worked (calling same `javac`) just a second before the failing call. I tried to reproduce it locally and also on a Codespaces instance, but never get the `Bad address` error to appear.

Do you have a better idea how to trace to the underlying issue on GHA?

>  If so, could we "just" `cp` everything from `JAVA_HOME_17_X64` to local path and try that?

I didn't try that out. Might work, too. Yet, there is an installation available at a local path: `bootjdk/jdk`.

> My concern with hooking jtreg to boot JDK would require jtreg to be buildable with STSes, like JDK 22 for eventual JDK 23 builds. Is there a plan/commitment for jtreg to do it?

Copied from https://github.com/openjdk/jtreg/blob/master/doc/building.md

> The script is intended to be run in a Unix-like shell, such as bash on Linux or Mac OS X, or with Cygwin, MSYS2, or WSL on Windows. At a minimum, you must either set the JAVA_HOME environment variable or specify the location of the JDK to be used to build jtreg with the --jdk command-line option. **It must be a recent build of JDK 11 or later.**

So yes, "JDK 11 or later" includes 22 and following releases.

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

PR Comment: https://git.openjdk.org/jdk/pull/14507#issuecomment-1594419317



More information about the build-dev mailing list