Re: [PATCH v4 1/1] git clone C:\cygwin\home\USER\repo' is working (again)

2019-05-02 Thread Achim Gratz
[Trying to revive that discussion] tbo...@web.de writes: > The cygwin layer "knows" that "C:\cygwin" is an absolute path, > but the new string operation does not. Then use the Cygwin API to produce the corresponding POSIX path and use that. Also, why does Git not use POSIX realpath on systems wh

Re: [PATCH v1/RFC 1/1] 'git clone C:\cygwin\home\USER\repo' is working (again)

2018-11-27 Thread Achim Gratz
Junio C Hamano writes: > I wonder if it makes the rest of the code simpler if we stripped > things like /cygdrive/c here exactly the sam way as we strip C: > For that, has_dos_drive_prefix() needs to know /cygdrive/[a-z], > which may not be a bad thing, I guess. Let's read on. The cygdrive prefix

Re: [PATCH v1/RFC 1/1] 'git clone C:\cygwin\home\USER\repo' is working (again)

2018-11-27 Thread Achim Gratz
tbo...@web.de writes: > The solution is to implement has_dos_drive_prefix(), skip_dos_drive_prefix() > is_dir_sep(), offset_1st_component() and convert_slashes() for cygwin > in the same way as it is done in 'Git for Windows' in compat/mingw.[ch] Please use the Cygwin API path conversion functions

Re: [PATCH] t/lib-httpd: switch SANITY check for NOT_ROOT

2015-01-16 Thread Achim Gratz
Kyle J. McKay writes: >>> "id -u" works for me in MSYS and cygwin (each appears to have it's >>> own id.exe). >> >> That's comforting. MSYS was the one I was most worried about. What UID >> do they report? I.e., do they correctly tell us if we are root (or >> more accurately, if we are not root)?