RFR: 8349665: Make clean removes module-deps.gmk [v2]

Magnus Ihse Bursie ihse at openjdk.org
Tue May 27 21:15:38 UTC 2025


On Tue, 27 May 2025 20:48:13 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> After [JDK-8292944](https://bugs.openjdk.org/browse/JDK-8292944), we only generate module-deps.gmk once at the start of Init.gmk. This causes `make clean <some other target>` to first generate and then remove module-deps.gmk after which the rest of the build proceeds without a module-deps.gmk, causing problems with dependencies.
>
> Magnus Ihse Bursie 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 three additional commits since the last revision:
> 
>  - Fix it for real
>  - Merge branch 'master' into fix-clean-module-deps
>  - 8349665: Make clean removes module-deps.gmk

Well it works, but causes a regression of JDK-8292944 ("make help" as the first target after configure). *sigh*

It turns out there had already been a partial regression of JDK-8292944 by JDK-8349515... This is now fixed as well. 

Third time's a charm! Now this should really, really work. (And we should really, really have a regression testing framework for the build system 😭).

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

PR Comment: https://git.openjdk.org/jdk/pull/24406#issuecomment-2914017583
PR Comment: https://git.openjdk.org/jdk/pull/24406#issuecomment-2914060430


More information about the build-dev mailing list