Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-09 Thread Andrew John Hughes
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-08 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-08 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-08 Thread Frederic Thevenet
On Thu, 7 Mar 2024 19:53:35 GMT, Andrew John Hughes wrote: >>> Also, it might be worth repeating one of my long-standing wishes: that the >>> version string should not be hard-coded into the build, but e.g. stored as >>> a string in the `release` file, and read from there. If we did that, the

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Julian Waters
On Thu, 7 Mar 2024 19:53:35 GMT, Andrew John Hughes wrote: > Also, it might be worth repeating one of my long-standing wishes: that the > version string should not be hard-coded into the build, but e.g. stored as a > string in the `release` file, and read from there. If we did that, the cost

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Erik Joelsson
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Andrew John Hughes
On Thu, 7 Mar 2024 17:32:02 GMT, Frederic Thevenet wrote: > Also, it might be worth repeating one of my long-standing wishes: that the > version string should not be hard-coded into the build, but e.g. stored as a > string in the `release` file, and read from there. If we did that, the cost

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Andrew John Hughes
On Thu, 7 Mar 2024 17:07:05 GMT, Magnus Ihse Bursie wrote: > There is an inherent conflict with creating a version string that is very > much up-to-date and includes ephemeral build data, and creating a robustly > reproducible build. If this patch were to remove HOTSPOT_BUILD_USER, and we >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Frederic Thevenet
On Thu, 7 Mar 2024 17:11:18 GMT, Magnus Ihse Bursie wrote: > Also, it might be worth repeating one of my long-standing wishes: that the > version string should not be hard-coded into the build, but e.g. stored as a > string in the `release` file, and read from there. If we did that, the cost

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Aleksey Shipilev
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-07 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 18:01:00 GMT, Aleksey Shipilev wrote: > > I agree that it is good to get rid of this. However, the reason it has > > stuck around for so long is, eh, that it has been around for so long, so it > > is a bit unclear what or who could be relying on this. > > The example I know

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread David Holmes
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Aleksey Shipilev
On Wed, 6 Mar 2024 17:17:28 GMT, Magnus Ihse Bursie wrote: > I agree that it is good to get rid of this. However, the reason it has stuck > around for so long is, eh, that it has been around for so long, so it is a > bit unclear what or who could be relying on this. The example I know is to

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 15:24:59 GMT, Andrew John Hughes wrote: >> I'm fine with this change. I wasn't around when this was introduced, but my >> guess is that it was relevant back when Hotspot and the rest of the JDK were >> often built separately. We have the username in the default $OPT string

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Magnus Ihse Bursie
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Andrew John Hughes
On Wed, 6 Mar 2024 13:57:37 GMT, Erik Joelsson wrote: > I'm fine with this change. I wasn't around when this was introduced, but my > guess is that it was relevant back when Hotspot and the rest of the JDK were > often built separately. We have the username in the default $OPT string for >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Erik Joelsson
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Andrew John Hughes
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

Re: RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Julian Waters
On Wed, 6 Mar 2024 11:57:27 GMT, Andrew John Hughes wrote: > The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the > internal version string since before the JDK was open-sourced, so the > reasoning for this is not in the public history. See >

RFR: 8327389: Remove use of HOTSPOT_BUILD_USER

2024-03-06 Thread Andrew John Hughes
The HotSpot code has inserted the value of `HOTSPOT_BUILD_USER` into the internal version string since before the JDK was open-sourced, so the reasoning for this is not in the public history. See https://github.com/openjdk/jdk/blob/master/src/hotspot/share/runtime/abstract_vm_version.cpp#L284