<div dir="ltr"><div>Thank you Severin, Goetz,</div><div><br></div><div>Yes agree. The intention is to at least make jdk17u tar ball reproducible with the exception of the CDS itself, which will remain for the stated reasons.</div><div><br></div><div>Could I request your review of the PR please? <a href="https://github.com/openjdk/jdk17u-dev/pull/1632">https://github.com/openjdk/jdk17u-dev/pull/1632</a></div><div><br></div><div>Many thanks</div><div>Andrew</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 7, 2023 at 3:07 PM Lindenmaier, Goetz <<a href="mailto:goetz.lindenmaier@sap.com">goetz.lindenmaier@sap.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="msg8960741765340745966">





<div style="overflow-wrap: break-word;" lang="EN-US">
<div class="m_8960741765340745966WordSection1">
<p class="MsoNormal">Hi,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I agree to what Severin says, this one seems acceptable. <u></u>
<u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">But I think the list mentioned by Ioi is definitely not a candidate for 17.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Best regards,<u></u><u></u></p>
<p class="MsoNormal">  Goetz<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-color:currentcolor currentcolor currentcolor blue;border-style:none none none solid;border-width:medium medium medium 1.5pt;padding:0cm 0cm 0cm 4pt">
<div>
<div style="border-color:rgb(225,225,225) currentcolor currentcolor;border-style:solid none none;border-width:1pt medium medium;padding:3pt 0cm 0cm">
<p class="MsoNormal"><b>From:</b> jdk-updates-dev <<a href="mailto:jdk-updates-dev-retn@openjdk.org" target="_blank">jdk-updates-dev-retn@openjdk.org</a>>
<b>On Behalf Of </b>Andrew Leonard<br>
<b>Sent:</b> Freitag, 4. August 2023 11:39<br>
<b>To:</b> <a href="mailto:jdk-updates-dev@openjdk.org" target="_blank">jdk-updates-dev@openjdk.org</a><br>
<b>Subject:</b> [jdk17u-dev] RFR: 8273092: Sort classlist in JDK image : regarding CDS archives<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">Hi,<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">I am currently backporting the fix to sort the classlist file, for a deterministic output:
<a href="https://github.com/openjdk/jdk17u-dev/pull/1632" target="_blank">https://github.com/openjdk/jdk17u-dev/pull/1632</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">and as Ioi Lam pointed out CDS archives in jdk17u are non-deterministic without various jdk-21+ fixes (<a href="https://bugs.openjdk.org/browse/JDK-8245335?jql=labels%20%3D%20reproducible-build%20AND%20labels%20%3D%20cds%20ORDER%20BY%20fixVersion%20ASC" target="_blank">https://bugs.openjdk.org/browse/JDK-8245335?jql=labels%20%3D%20reproducible-build%20AND%20labels%20%3D%20cds%20ORDER%20BY%20fixVersion%20ASC</a>),
 which are unlikely to be backported as jdk17u CDS is quite different to 21+ now.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">However, it would still be useful to at least have the classlist deterministic, to achieve a JDK tarball that is reproducible bar the CDS archives. There is an argument that CDS archives could be produced at a different point in time from
 the build tarball, for example maybe via the installer at user install time.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thoughts please? on whether it is useful to merge this classlist sorting, regardless of the CDS archive issue?<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thanks<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Andrew<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div>
</div>
</div>

</div></blockquote></div>