[jdk17u-dev] RFR: 8273092: Sort classlist in JDK image

Ioi Lam iklam at openjdk.org
Thu Aug 3 15:50:43 UTC 2023


On Wed, 2 Aug 2023 21:47:24 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Hi @iklam , would you be able to review this backport of your original classlist sorting please? thanks
>
>> Hi @iklam , would you be able to review this backport of your original classlist sorting please? thanks
> 
> Hmm, I am not sure why this backport is necessary.

> > > Hi @iklam , would you be able to review this backport of your original classlist sorting please? thanks
> > 
> > 
> > Hmm, I am not sure why this backport is necessary.
> 
> @iklam The classlist file ordering is seemingly random, and thus 2 builds of the same code produces classlist files that differ.

Is the goal for reproducible builds? Backporting this change alone isn't enough. The following are needed:

https://bugs.openjdk.org/issues/?jql=labels%20%3D%20reproducible-build%20AND%20labels%20%3D%20cds%20ORDER%20BY%20fixVersion%20ASC

However, the CDS code has changed significantly since JDK 17, so backporting will be very difficult.

Please follow https://wiki.openjdk.org/display/JDKUpdates/How+to+contribute+or+backport+a+fix and have a discussion on the [jdk-updates-dev mailing list](https://mail.openjdk.org/mailman/listinfo/jdk-updates-dev) before proceeding.

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1632#issuecomment-1664227750


More information about the jdk-updates-dev mailing list