This is an attempt to fix the issue on Windows when no cygwin Git is installed 
or the Git for Windows installation has precedence in PATH lookup. The path to 
the Windows GIT installation usually resides in `C:\Program Files` which 
contains a space and thus needs some special handling.

There exists code in `UTIL_LOOKUP_PROGS`/`UTIL_FIXUP_EXECUTABLE` that would 
handle this. However, it relies on initializations made in 
`PLATFORM_SETUP_OPENJDK_BUILD_AND_TARGET` and `BASIC_SETUP_PATHS`. Currently, 
`UTIL_LOOKUP_PROGS(GIT, git)` is called too early in configure and hence the 
problematic Windows path is not handled correctly. My fix makes sure that only 
the barely necessary tool lookups required for 
`PLATFORM_SETUP_OPENJDK_BUILD_AND_TARGET` and `BASIC_SETUP_PATHS` are made in 
`BASIC_SETUP_FUNDAMENTAL_TOOLS` and everything else is moved into another macro 
called `BASIC_SETUP_TOOLS` that is invoked after path handling is set up 
correctly, which includes the lookup of git.

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

Commit messages:
 - JDK-8289735

Changes: https://git.openjdk.org/jdk/pull/13629/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=13629&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8289735
  Stats: 34 lines in 2 files changed: 21 ins; 8 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/13629.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13629/head:pull/13629

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

Reply via email to