Integrated: 8369242: Rename URL variables in devkit/Tools.gmk

Mikael Vidstedt mikael at openjdk.org
Tue Oct 7 00:56:00 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

This pull request has now been integrated.

Changeset: eb34a117
Author:    Mikael Vidstedt <mikael at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/eb34a117934951af075a425ce2cf8d3b1ced9700
Stats:     10 lines in 1 file changed: 0 ins; 0 del; 10 mod

8369242: Rename URL variables in devkit/Tools.gmk

Reviewed-by: erikj

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

PR: https://git.openjdk.org/jdk/pull/27661


More information about the build-dev mailing list