RFR: 7903720: Change layout of jextract image [v4]
Jorn Vernee
jvernee at openjdk.org
Tue Apr 30 17:09:31 UTC 2024
> See JBS issue for problem description.
>
> This patch changes the layout of the jextract image we produce. The new image has the 2 launcher scripts in the root of the image, with a nested `runtime` folder containing the jlink runtime image.
>
> I've added both launcher scripts to the repo under the `src/main` directory, and they are copied to the image during the build. The script is somewhat simplified by removing a blank variable that was not being used. I also took the opportunity to address an issue with the windows launcher script turning off `echo`. Instead it now avoids output using the `@` prefix for each command in the script.
>
> I've tested that the scripts work (using `jextract --help`) on both Linux and Windows.
Jorn Vernee has updated the pull request incrementally with four additional commits since the last revision:
- set mac version to x64 pt2
- set mac version to x64
- only set file permissions if supported
- pass JEXTRACT_JAVA_OPTIONS to launcher
-------------
Changes:
- all: https://git.openjdk.org/jextract/pull/241/files
- new: https://git.openjdk.org/jextract/pull/241/files/89b8e2bd..a2768cfb
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jextract&pr=241&range=03
- incr: https://webrevs.openjdk.org/?repo=jextract&pr=241&range=02-03
Stats: 13 lines in 4 files changed: 2 ins; 0 del; 11 mod
Patch: https://git.openjdk.org/jextract/pull/241.diff
Fetch: git fetch https://git.openjdk.org/jextract.git pull/241/head:pull/241
PR: https://git.openjdk.org/jextract/pull/241
More information about the jextract-dev
mailing list