> This PR addresses an issue that can be observed when building on Windows with 
> configure options `--enable-linkable-runtime` and 
> `--with-external-symbols-in-bundles=public`.
> 
> The problem is that with this special build configuration, we build two sets 
> of .pdb files for the binaries. The first set is the standard debug symbols 
> files named <binary-name>.pdb. The second set consists of stripped debug 
> symbols file called <binary-name>.stripped.pdb which have less information 
> but enough to present line numbers in hs-err files.
> 
> During build we use the *.stripped.pdb files for compiling the jmods and also 
> the bundle files. However, in the images folder, both sets of .pdb files 
> exist. The tests for runtime linking will now, in the absence of jmod files, 
> pick up the .pdb files (without *stripped*) from images, but in the runtime 
> the hashes of the *stripped* files are stored.
> 
> With this change, the standard .pdb files in the 
> `--with-external-symbols-in-bundles=public` configuration are now the 
> stripped files and we create a set of full pdb files named *.full.pdb. Jmods 
> and Bundles still contain the stripped pdb files and we also fix the issue 
> that the debug symbols bundle also contained stripped pdb files so far. With 
> this fix, it will contain the full pdb files and extracting these over a JDK 
> runtime will replace stripped pdbs with full pdbs.

Christoph Langer has updated the pull request with a new target base due to a 
merge or a rebase. The incremental webrev excludes the unrelated changes 
brought in by the merge/rebase. The pull request contains four additional 
commits since the last revision:

 - Merge branch 'master' into JDK-8351842
 - Merge branch 'master' into JDK-8351842
 - Fix tests
 - JDK-8351842

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24012/files
  - new: https://git.openjdk.org/jdk/pull/24012/files/f560fd21..e8bff098

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24012&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24012&range=01-02

  Stats: 942724 lines in 12132 files changed: 498672 ins; 368027 del; 76025 mod
  Patch: https://git.openjdk.org/jdk/pull/24012.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24012/head:pull/24012

PR: https://git.openjdk.org/jdk/pull/24012

Reply via email to