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.1558-1-tbo...@web.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

Reply via email to