Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-04-07 Thread Severin Gehwolf
On Tue, 1 Apr 2025 16:12:39 GMT, Alan Bateman wrote: > > @AlanBateman Any more thoughts on this? We'd need to include a patch like > > this one for getting the Fedora JDK 24+ builds to work with JEP 493 > > enabled. Thanks! > > Allowing for a small number of upgradable files is needed, I see y

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-04-05 Thread Severin Gehwolf
On Wed, 26 Mar 2025 10:12:43 GMT, Alan Bateman wrote: > > The cacerts issue mentioned in the JBS issue relates to an RPM installation > > of the JDK where the cacerts file is a symlink to the distro provided one. > > So I think that's "use system" issue. > > TZ updates would potentially break t

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-04-05 Thread Severin Gehwolf
On Wed, 26 Mar 2025 17:43:50 GMT, Christoph Langer wrote: > Would it maybe make sense/be possible to offer some re-hash functionality for > using in 2nd step builds? What would that be? Right now linking from the run-time image doesn't allow for `jdk.jlink` to be included, which prevents a 2nd

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-04-01 Thread Alan Bateman
On Wed, 26 Mar 2025 14:50:11 GMT, Alan Bateman wrote: >>> > The cacerts issue mentioned in the JBS issue relates to an RPM >>> > installation of the JDK where the cacerts file is a symlink to the distro >>> > provided one. So I think that's "use system" issue. >>> > TZ updates would potentially

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-04-01 Thread Severin Gehwolf
On Wed, 26 Mar 2025 14:50:11 GMT, Alan Bateman wrote: >>> > The cacerts issue mentioned in the JBS issue relates to an RPM >>> > installation of the JDK where the cacerts file is a symlink to the distro >>> > provided one. So I think that's "use system" issue. >>> > TZ updates would potentially

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v3]

2025-03-28 Thread Severin Gehwolf
On Wed, 26 Mar 2025 17:09:06 GMT, Severin Gehwolf wrote: >> Please review this enhancement which adds a hidden `jlink` option >> `--sha-overrides` that can be used to provide alternative hash sums for >> files in an image. Please see the bug for use-cases as to why this is >> needed. This patc

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-27 Thread Severin Gehwolf
On Wed, 26 Mar 2025 13:42:11 GMT, Severin Gehwolf wrote: >> test/jdk/tools/jlink/runtimeImage/ModifiedFilesWithShaOverrideTest.java line >> 32: >> >>> 30: * @requires (vm.compMode != "Xcomp" & os.maxMemory >= 2g & os.family >>> == "linux") >>> 31: * @library ../../lib /test/lib >>> 32: * @e

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Severin Gehwolf
On Wed, 26 Mar 2025 08:28:23 GMT, Christoph Langer wrote: >> Severin Gehwolf has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Copyright updates >> - Allow for ${java.home} substitution when \@file is being passed >> >>Also refa

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v3]

2025-03-26 Thread Severin Gehwolf
> Please review this enhancement which adds a hidden `jlink` option > `--sha-overrides` that can be used to provide alternative hash sums for files > in an image. Please see the bug for use-cases as to why this is needed. This > patch allows for the `--sha-overrides` option to be either specifie

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Christoph Langer
On Wed, 26 Mar 2025 17:47:07 GMT, Severin Gehwolf wrote: > > Would it maybe make sense/be possible to offer some re-hash functionality > > for using in 2nd step builds? > > What would that be? Right now linking from the run-time image doesn't allow > for `jdk.jlink` to be included, which preve

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Christoph Langer
On Wed, 26 Mar 2025 14:50:11 GMT, Alan Bateman wrote: > > I'll keep looking into this specific case. However, it sounds a bit > > orthogonal to the patch at hand which I do believe we still need for the > > original reasons mentioned (RPM changing binaries after the JDK build is > > long done

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Alan Bateman
On Wed, 26 Mar 2025 14:42:12 GMT, Severin Gehwolf wrote: > I'll keep looking into this specific case. However, it sounds a bit > orthogonal to the patch at hand which I do believe we still need for the > original reasons mentioned (RPM changing binaries after the JDK build is long > done and t

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Christoph Langer
On Tue, 25 Mar 2025 17:43:07 GMT, Severin Gehwolf wrote: >> Please review this enhancement which adds a hidden `jlink` option >> `--sha-overrides` that can be used to provide alternative hash sums for >> files in an image. Please see the bug for use-cases as to why this is >> needed. This patc

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Alan Bateman
On Wed, 26 Mar 2025 09:53:51 GMT, Severin Gehwolf wrote: > The cacerts issue mentioned in the JBS issue relates to an RPM installation > of the JDK where the cacerts file is a symlink to the distro provided one. So > I think that's "use system" issue. > > TZ updates would potentially break thi

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Severin Gehwolf
On Wed, 26 Mar 2025 08:53:21 GMT, Alan Bateman wrote: > > @AlanBateman Would we need a CSR for this? This isn't any option that shows > > up anywhere user-visible, so I'm thinking not. Please let me know. Thanks! > > There isn't any change to a supported interface so probably not. Thanks. > T

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-26 Thread Alan Bateman
On Tue, 25 Mar 2025 17:54:43 GMT, Severin Gehwolf wrote: > @AlanBateman Would we need a CSR for this? This isn't any option that shows > up anywhere user-visible, so I'm thinking not. Please let me know. Thanks! There isn't any change to a supported interface so probably not. That said, I don'

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-25 Thread Severin Gehwolf
> Please review this enhancement which adds a hidden `jlink` option > `--sha-overrides` that can be used to provide alternative hash sums for files > in an image. Please see the bug for use-cases as to why this is needed. This > patch allows for the `--sha-overrides` option to be either specifie

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-25 Thread Severin Gehwolf
On Tue, 25 Mar 2025 17:43:07 GMT, Severin Gehwolf wrote: >> Please review this enhancement which adds a hidden `jlink` option >> `--sha-overrides` that can be used to provide alternative hash sums for >> files in an image. Please see the bug for use-cases as to why this is >> needed. This patc

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-25 Thread Severin Gehwolf
On Tue, 25 Mar 2025 17:43:07 GMT, Severin Gehwolf wrote: >> Please review this enhancement which adds a hidden `jlink` option >> `--sha-overrides` that can be used to provide alternative hash sums for >> files in an image. Please see the bug for use-cases as to why this is >> needed. This patc

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image [v2]

2025-03-25 Thread Severin Gehwolf
On Tue, 25 Mar 2025 13:51:10 GMT, Christoph Langer wrote: > The only review thing I could find were the copyright years which need > updates. I've fixed copyright years in the latest update. - PR Comment: https://git.openjdk.org/jdk/pull/24190#issuecomment-2752051896

Re: RFR: 8352689: Allow for hash sum overrides when linking from the run-time image

2025-03-25 Thread Christoph Langer
On Mon, 24 Mar 2025 11:02:15 GMT, Severin Gehwolf wrote: > Please review this enhancement which adds a hidden `jlink` option > `--sha-overrides` that can be used to provide alternative hash sums for files > in an image. Please see the bug for use-cases as to why this is needed. This > patch al

RFR: 8352689: Allow for hash sum overrides when linking from the run-time image

2025-03-24 Thread Severin Gehwolf
Please review this enhancement which adds a hidden `jlink` option `--sha-overrides` that can be used to provide alternative hash sums for files in an image. Please see the bug for use-cases as to why this is needed. This patch allows for the `--sha-overrides` option to be either specified multip