Re: [PATCH 0/2] Fix warnings on access of a remote with Windows paths

From: Johannes Sixt <j6t@kdbg.org>
Date: May 22, 2017, 16:36 UTC
Message-ID: <947b2453-52b6-4940-fd20-ddf350c5df4d@kdbg.org>
In-Reply-To: <alpine.DEB.2.21.1.1705221357360.3610@virtualbox>

Am 22.05.2017 um 13:59 schrieb Johannes Schindelin:

Show 16 quoted lines

> On Sat, 20 May 2017, Johannes Sixt wrote:
>> This small series fixes these warnings on Windows:
>>
>> C:\Temp\gittest>git fetch C:\Temp\gittest
>> warning: unable to access '.git/remotes/C:\Temp\gittest': Invalid argument
>> warning: unable to access '.git/branches/C:\Temp\gittest': Invalid argument
>>  From C:\Temp\gittest
>>   * branch            HEAD       -> FETCH_HEAD
>>
>> The fix is in the second patch; the first patch is a
>> preparation that allows to write the fix in my preferred style.
> 
> Thank you!
> 
> Maybe you want to accompany these patches with a simple test case that
> uses e.g. ls-remote on $(pwd | tr / \\\\)?

Actually, no, I don't want to. It would have to be protected by MINGW, and I don't want to burden us (and here I mean Windows folks) with a check for a cosmetic deficiency. (Shell scripts, slow forks, yadda, yadda...)

-- Hannes