[jdk8u-dev] RFR: 8281814: Debuginfo.diz contains redundant build path after backport JDK-8025936
Andrew John Hughes
andrew at openjdk.java.net
Fri May 13 03:16:58 UTC 2022
On Wed, 30 Mar 2022 07:45:08 GMT, Dongbo He <dongbohe at openjdk.org> wrote:
> Hi,
>
> I would like to backport 8035134 to fix debuginfo regression, introduced by [JDK-8025936](https://bugs.openjdk.java.net/browse/JDK-8025936).
> As discussed on [JDK-8281814](https://bugs.openjdk.java.net/browse/JDK-8281814), we cannot see the details of the original patch.
> It is only a different context, no risk.
>
> Testing: worked correctly after patch.
It looks like it is largely reverted again in [this 11u commit](https://github.com/openjdk/jdk11u-dev/commit//a350f3bda730bc3276e#diff-805224b793cfe47bf6bebf62286f34adb363f0352a9dac3689825b823886e534L882). I'll look at backporting just the changes to that file, as the rest is intrusive or not relevant.
On Mac, `$1_DEBUGINFO_FILES` contains just a directory, `$$($1_OBJECT_DIR)/$$($1_BASENAME).dSYM` and so ends up empty when this is filtered out. That patch changes the MacOS setup of `$1_DEBUGINFO_FILES` to refer to files.
It's not clear to me why this `notdir` fix was applied in the first place. The `subst` change in the later commit was identified in [JDK-8281814](https://bugs.openjdk.java.net/browse/JDK-8281814) but not applied.
-------------
PR: https://git.openjdk.java.net/jdk8u-dev/pull/26
More information about the jdk8u-dev
mailing list