DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:to:references:subject
	:content-type; q=dns; s=default; b=IXkT/hG63SNM2XqmNmqMtTZYTvjo4
	+3e+TWcbqU4+N6fR4yPcATdsg+I1rMsNtomlDFfwmyenaI9+WF4rAjc05IAmkRdv
	dIW0i3mDQksgNdF897cYiu+fGGxL1ruHC4so80f7TvV4LmtazwBV5p/8padPukap
	OUWcLlKR+hCy7w=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Message-ID: <5bfc994b.1c69fb81.2bacb.e7da@mx.google.com>
Date: Mon, 26 Nov 2018 17:09:31 -0800 (PST)
From: Steven Penny <svnpenn AT gmail DOT com>
To: cygwin AT cygwin DOT com
References: <CA+kUOamKzp=jjKc1OLGo1vmepoee5ABwgUpYJF9Lq1CJjOBy9Q AT mail DOT gmail DOT com>
Subject: Re: Cygwin Git with Windows paths
Content-Type: text/plain; charset=utf8; format=flowed
User-Agent: Tryst/2.8.0 (cup.github.io/umber/awk/tryst)

On Mon, 26 Nov 2018 22:54:14, Adam Dinwoodie wrote:
> Personally, I don't see this as a bug; AIUI using Windows style paths
> isn't something that is supported in general in Cygwin, even if it's
> something that works in some circumstances.

It is a bug. Even when you use Unix paths, Cygwin is doing path conversions:

    $ ls /var
    cache  lib  log  run  tmp

    $ strace ls /var | grep -E '(conv_to|normalize)_(posix|win32)_path' | wc
    32     320    3337

So either this code should be pulled out of the Cygwin DLL, or people should
stop saying that its not supported.

> I see you've raised this on the Git mailing list as well, and if the
> upstream Git package starts to handle such paths, I'll take the
> relevant patches. However since I don't consider this a bug, I'm not
> going to raise it myself.

Its not fixed upstream (yet), but a patch is available that fixes the issue:

http://public-inbox.org/git/20181126173252 DOT 1558-1-tboegi AT web DOT de

Note carefully that Windows path handling previously worked with Cygwin Git
until Dec 2016, when a patch was introduced that broke that use case. The patch
in question repairs the path handling so that Unix and Windows paths are both
supported with Cygwin Git again.


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple