On Mon, 24 Apr 2023 21:44:00 GMT, Christoph Langer <clan...@openjdk.org> wrote:
>> 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. > > Christoph Langer has updated the pull request incrementally with one > additional commit since the last revision: > > Adjust comments Marked as reviewed by mbaesken (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/13629#pullrequestreview-1399328940