On Mon, 6 Oct 2025 21:44:33 GMT, Mikael Vidstedt <[email protected]> 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

Reply via email to