On Fri, 14 Mar 2025 12:29:22 GMT, Christoph Langer <[email protected]> wrote:
> Alternative approach to #24012
>
> This keeps the current handling of *.pdb vs *.stripped.pdb which allows
> debugging at the cost of a little hack in jlink. Maybe the code in jlink can
> be improved, e.g. make it more conditional.
>
> I'm running this through our testing still to see whether it'll resolve all
> of the test issues and does not introduce regressions.
src/jdk.jlink/share/classes/jdk/tools/jlink/internal/JRTArchive.java line 226:
> 224: if (Files.exists(strippedPath)) {
> 225: path = strippedPath;
> 226: }
This is super weird to put here. I could imagine introducing a Windows specific
plugin or option for jlink to help with this setup but I don't think we should
rush into a change to JRTArchive to workaround this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24057#discussion_r1995601023
