Warning about git from 'make test' on Windows

[Bradford Wetmore]

Andrey/Christoph,

This may be a little late, but...

> Maybe this gives a hint on how to fix it?

I (and others in my group) typically use the root c:/cygwin and avoid 
this whole issue.

Brad



On 6/3/2022 12:09 AM, Langer, Christoph wrote:
> Hi,
> 
> I see the same on my windows build. I verified that 8.3 filenames are enabled.
> 
> I took a little closer look. In my PATH env in Cygwin I have: ...:/cygdrive/c/Program Files/Git/cmd:...
> 
> The git path is resolved via UTIL_LOOKUP_PROGS(GIT, git) in basic_tools.m4. UTIL_LOOKUP_PROGS, defined in util_paths.m4 must somehow not be capable to correctly resolve the path with spaces. The output in configure is:
> 
> checking for git...  Files/Git/cmd/git
> 
> Maybe this gives a hint on how to fix it? I guess I can configure my path to use the 8.3 notation. But maybe UTIL_LOOKUP_PROGS could be improved?
> 
> Cheers
> Christoph
> 
>> -----Original Message-----
>> From: build-dev <build-dev-retn at openjdk.java.net> On Behalf Of Magnus Ihse
>> Bursie
>> Sent: Freitag, 3. Juni 2022 01:10
>> To: Andrey Turbanov <turbanoff at gmail.com>; build-dev at openjdk.java.net
>> Subject: Re: Warning about git from 'make test' on Windows
>>
>>
>> On 2022-06-02 21:26, Andrey Turbanov wrote:
>>> Hello.
>>> I noticed strange warnings produced by 'make test' recently on my
>>> Win11 installation. For example:
>>>
>>> $ make test TEST="tier1"
>>> Building target 'test' in configuration 'windows-x86_64-server-release'
>>> /usr/bin/bash: Files/Git/cmd/git: No such file or directory
>>> /usr/bin/bash: Files/Git/cmd/git: No such file or directory Test
>>> selection 'tier1', will run:
>>>
>>>
>>> Also found this in
>>> build\windows-x86_64-server-release\configure-support\config.log
>>>
>>> configure:29274: checking for git
>>> configure:29502: result:  Files/Git/cmd/git
>>>
>>> Seems like space in git path is incorrectly handled.
>>> Is it a known issue or is it a limitation of autoconf?
>> Spaces in paths is always tricky and is best avoided. That being said, we try to
>> make autoconf robust for tools in directories with space in them.
>>
>> Are you running in Cygwin?
>>
>> Can you verify that your C: drive can use short paths. See
>> https://openjdk.java.net/groups/build/doc/building.html#spaces-in-path
>>
>> /Magnus
>>>
>>> Andrey Turbanov
>