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

Erik Joelsson erikj 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

make/Main.gmk line 46:

> 44: include FindTests.gmk
> 45: 
> 46: ifneq ($(UPDATE_MODULE_DEPS), )

Why not check equals true?
Suggestion:

ifeq ($(UPDATE_MODULE_DEPS), true)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24406#discussion_r2110228876


More information about the build-dev mailing list