RFR: 8345424: Move FindDebuginfoFiles out of FileUtils.gmk

Erik Joelsson erikj at openjdk.org
Thu Dec 5 14:01:39 UTC 2024


On Thu, 5 Dec 2024 09:55:28 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> After [JDK-8339480](https://bugs.openjdk.org/browse/JDK-8339480), tier5 tests started failing in the `special:make:all` test. The culprit turned out to be that FindDebuginfoFiles was added to FileUtils.gmk, causing the find cache to be populated when it shouldn't be.
> 
> As a simple fix I am moving FindDebuginfoFiles into a separate file. The `DebugInfoUtils.gmk` is a bit odd since it is so specialized, and only shared among two files, but I could find no better way. The code is still a bit too complex to be duplicated, and I could not find any other place where it would belong better. The handling of debuginfo files is a bit spread out in the code base, so maybe this opens up for the chance to consolidate more debuginfo handling into this file further on.

Marked as reviewed by erikj (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22569#pullrequestreview-2481788124


More information about the build-dev mailing list