RFR: 8358515: make cmp-baseline is broken after JDK-8349665
Erik Joelsson
erikj at openjdk.org
Tue Jun 3 18:16:17 UTC 2025
On Tue, 3 Jun 2025 14:56:32 GMT, Magnus Ihse Bursie <ihse at 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
More information about the build-dev
mailing list