Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-04-04 Thread Severin Gehwolf
On Thu, 14 Mar 2024 17:46:19 GMT, Erik Joelsson wrote: >> Severin Gehwolf has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix comment in autoconf file > > make/Images.gmk line 126: > >> 124: RL_BUILD_MODULE_NAME :=

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-18 Thread Erik Joelsson
On Mon, 18 Mar 2024 18:40:36 GMT, Severin Gehwolf wrote: > What I mean to convey is that how the directory tree in `images/jdk` looks > like shouldn't change based on the `--enable-linkable-runtime-image` > configure flag. That is, there will be a `images/jdk/jmods` folder unless >

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-18 Thread Severin Gehwolf
On Mon, 18 Mar 2024 18:08:23 GMT, Mandy Chung wrote: > > Yes we do. The main reason being that the `jdk` image has more to it than > > just the image. `src.zip`, CDS data, `demo` and so on. We don't want to > > duplicate that. To us, including the `jmods` folder is something that comes > >

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-18 Thread Mandy Chung
On Mon, 18 Mar 2024 13:01:10 GMT, Severin Gehwolf wrote: > Yes we do. The main reason being that the `jdk` image has more to it than > just the image. `src.zip`, CDS data, `demo` and so on. We don't want to > duplicate that. To us, including the `jmods` folder is something that comes > into

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-18 Thread Severin Gehwolf
On Fri, 15 Mar 2024 18:09:36 GMT, Mandy Chung wrote: > > Wasn't the intention to make "creating a linkable runtime image" a build > > only decision and make the relevant infrastructure classes build-only > > artefacts? > > Build-only decision means that the linkable runtime image is only

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-18 Thread Severin Gehwolf
On Fri, 15 Mar 2024 19:29:49 GMT, Mandy Chung wrote: > > > If `--enable-runtime-link-image` is enabled, the JDK image does not > > > include the packaged modules. > > > > > > That's not true. Right now `--enable-runtime-link-image` modifies how the > > `lib/modules` image looks like (adds a

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-16 Thread Mandy Chung
On Fri, 15 Mar 2024 10:53:29 GMT, Severin Gehwolf wrote: > Wasn't the intention to make "creating a linkable runtime image" a build only > decision and make the relevant infrastructure classes build-only artefacts? Build-only decision means that the linkable runtime image is only produced by

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-16 Thread Mandy Chung
On Fri, 15 Mar 2024 09:55:15 GMT, Severin Gehwolf wrote: > > If `--enable-runtime-link-image` is enabled, the JDK image does not include > > the packaged modules. > > That's not true. Right now `--enable-runtime-link-image` modifies how the > `lib/modules` image looks like (adds a bunch of

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-15 Thread Severin Gehwolf
On Thu, 14 Mar 2024 20:54:32 GMT, Mandy Chung wrote: > > The updated patch uses a **build-only** `jlink` plugin, still called > > `--create-linkable-runtime` which is _a)_ added only at build time with > > `--add-modules` and _b)_ now generates the diff and prepares the image for > > runtime

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-15 Thread Severin Gehwolf
On Thu, 14 Mar 2024 21:16:04 GMT, Erik Joelsson wrote: > About the configure options, > > ``` > --enable-keep-packaged-modules > enable keeping of packaged modules in jdk image > [enabled] > --enable-runtime-link-image > enable producing

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Magnus Ihse Bursie
On Thu, 14 Mar 2024 14:24:57 GMT, Severin Gehwolf wrote: >> Please review this patch which adds a jlink mode to the JDK which doesn't >> need the packaged modules being present. A.k.a run-time image based jlink. >> Fundamentally this patch adds an option to use `jlink` even though your JDK >>

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Erik Joelsson
On Thu, 14 Mar 2024 20:54:32 GMT, Mandy Chung wrote: > About the configure options, > > ``` > --enable-keep-packaged-modules > enable keeping of packaged modules in jdk image > [enabled] > --enable-runtime-link-image > enable producing an

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Mandy Chung
On Thu, 14 Mar 2024 14:24:57 GMT, Severin Gehwolf wrote: >> Please review this patch which adds a jlink mode to the JDK which doesn't >> need the packaged modules being present. A.k.a run-time image based jlink. >> Fundamentally this patch adds an option to use `jlink` even though your JDK >>

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Erik Joelsson
On Thu, 14 Mar 2024 14:27:47 GMT, Severin Gehwolf wrote: >> Severin Gehwolf has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix comment in autoconf file > > make/Images.gmk line 33: > >> 31: include Modules.gmk >> 32: include Utils.gmk

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Erik Joelsson
On Thu, 14 Mar 2024 14:24:57 GMT, Severin Gehwolf wrote: >> Please review this patch which adds a jlink mode to the JDK which doesn't >> need the packaged modules being present. A.k.a run-time image based jlink. >> Fundamentally this patch adds an option to use `jlink` even though your JDK >>

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Severin Gehwolf
On Thu, 14 Mar 2024 14:24:57 GMT, Severin Gehwolf wrote: >> Please review this patch which adds a jlink mode to the JDK which doesn't >> need the packaged modules being present. A.k.a run-time image based jlink. >> Fundamentally this patch adds an option to use `jlink` even though your JDK >>

Re: RFR: 8311302: Allow for jlinking a custom runtime without packaged modules being present [v20]

2024-03-14 Thread Severin Gehwolf
> Please review this patch which adds a jlink mode to the JDK which doesn't > need the packaged modules being present. A.k.a run-time image based jlink. > Fundamentally this patch adds an option to use `jlink` even though your JDK > install might not come with the packaged modules (directory