RFR: 8369242: Rename URL variables in devkit/Tools.gmk
Erik Joelsson
erikj at openjdk.org
Mon Oct 6 22:20:45 UTC 2025
On Mon, 6 Oct 2025 21:44:33 GMT, Mikael Vidstedt <mikael at openjdk.org> wrote:
> Background (from JBS):
>
> To keep us on our toes the source bundle URL variables in Tools.gmk are simply named after the corresponding dependency. For example (from Tools.gmk):
>
>
> GCC := http://ftp.gnu.org/pub/gnu/gcc/$(gcc_ver)/$(gcc_ver).tar.xz
>
>
> There's also a lower case variable set up for the "done" marker file for each dependency (from Tools.gmk):
>
>
> # Define marker files for each source package to be compiled
> $(foreach dep,$(dependencies),$(eval $(dep) = $(TARGETDIR)/$($(dep)_ver).done))
>
>
> Will do something equivalent to:
>
>
> gcc = target/dir/gcc-14.2.0.done
>
>
> Let's rename the former variable to make it clearer that is, in fact, a URL.
>
>
> Description:
>
> The change renames the variables in question to include a `_URL` suffix and updates the uses accordingly.
>
> Testing:
>
> * Built devkit on linux-x64
> * Verified that the JDK builds with the new devkit
Marked as reviewed by erikj (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/27661#pullrequestreview-3307687056
More information about the build-dev
mailing list