On Tue, 3 Jun 2025 14:56:32 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

> The fix in to [JDK-8357991](https://bugs.openjdk.org/browse/JDK-8357991) was 
> not enough to solve all problems that had been introduced with JDK-8349665. 
> Also builds run using `COMPARE_BUILD` was affected.
> 
> The core of the problem was that there were multiple additional ways to run 
> what esstentially amount to `make -f Init.gmk main`, in places that I did not 
> find before. So I took a step back and chose a safer approach: now the `main` 
> target in `Init.gmk` depends on an additional target, that will always make 
> sure that the necessary helper files are present. This means the specific 
> patch for bootcycle builds in JDK-8357991 could be reverted. This solution is 
> also future-proof if we should figure out any new creative ways of calling 
> `main` in `Init.gmk`.

Marked as reviewed by erikj (Reviewer).

make/Init.gmk line 121:

> 119:          $(MAKE) $(MAKE_ARGS) -j 1 -f make/Main.gmk $(USER_MAKE_VARS) \
> 120:          UPDATE_MODULE_DEPS=true NO_RECIPES=true \
> 121:          create-main-targets-include )

That looks like a bit of extra indentation.

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

PR Review: https://git.openjdk.org/jdk/pull/25618#pullrequestreview-2893553379
PR Review Comment: https://git.openjdk.org/jdk/pull/25618#discussion_r2124559367

Reply via email to