Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread Pavel Kudrna
Corinna Vinschen wrote: On Jul 13 14:19, Pavel Kudrna wrote: Novell client uses paths of type z:. as so called search drives. It is stupid because if works only until you change current dir on that drive. It should use z:\ instead but we can't change it and it works. So the good is that it

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread Matthew Woehlke
(nonstandard quoting reformatted) André Bleau wrote: Corinna Vinschen wrote: (*) and, funny enough, there's no API call in Win32 to return a DCWD. There's just a call GetCurrentDirectory() which returns *the* CWD. Which makes sense, given that the RTL_USER_PROCESS_PARAMETERS can only store one

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread Corinna Vinschen
On Jul 13 14:19, Pavel Kudrna wrote: > Corinna Vinschen wrote: > >I don't see how converting a pure DOS-ism > >into a wrong POSIX path is doing any good. > > > Novell client uses paths of type z:. as so called search drives. It is > stupid because if works > only until you change current dir on

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread André Bleau
Corinna Vinschen wrote: [...] (*) and, funny enough, there's no API call in Win32 to return a DCWD. There's just a call GetCurrentDirectory() which returns *the* CWD. Which makes sense, given that the RTL_USER_PROCESS_PARAMETERS can only store one CWD per process. Well, this is no

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread Pavel Kudrna
Corinna Vinschen wrote: On Jul 12 21:01, Pavel Kudrna wrote: After that change cygwin_conv_to_posix_path() treats all following paths in the same way: c:\ /cygdrive/c c: /cygdrive/c c:\.\ /cygdrive/c/ c:\./cygdrive/c c:.\/cygdrive/c/./ c:. /cygdrive/c/. Before the sugg

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-13 Thread Corinna Vinschen
On Jul 12 21:01, Pavel Kudrna wrote: > After that change cygwin_conv_to_posix_path() treats all following paths > in the same > way: > c:\ /cygdrive/c > c: /cygdrive/c > c:\.\ /cygdrive/c/ > c:\./cygdrive/c > c:.\/cygdrive/c/./ > c:. /cygdrive/c/. > Before the suggested chan

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-12 Thread Pavel Kudrna
Christopher Faylor wrote: The issue is well understood and there is no reason to send more email about this unless the email contains a patch. cgf The current isabspath() macro requires an absolute path starting with drive letter and colon to be just drive letter and colon or to be followed

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-05 Thread Christopher Faylor
On Thu, Jul 05, 2007 at 06:50:45PM -0400, Larry Hall (Cygwin) wrote: > Pavel Kudrna wrote: >>Christopher Faylor wrote: >>>I suppose it is possible that we'd consider a nicely written, minimally >>>intrusive patch to use these but I'm not 100% certain that it is even >>>possible to do this cleanly g

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-05 Thread Larry Hall (Cygwin)
Pavel Kudrna wrote: Christopher Faylor wrote: I suppose it is possible that we'd consider a nicely written, minimally intrusive patch to use these but I'm not 100% certain that it is even possible to do this cleanly given the way the environment variable/path conversion handling works. Lets

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-05 Thread Pavel Kudrna
Christopher Faylor wrote: I suppose it is possible that we'd consider a nicely written, minimally intrusive patch to use these but I'm not 100% certain that it is even possible to do this cleanly given the way the environment variable/path conversion handling works. Lets look back to original

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-04 Thread Christopher Faylor
On Wed, Jul 04, 2007 at 11:16:43AM +0200, Pavel Kudrna wrote: >Do you mean that the variables are present but hidden in that case? It depends on what you mean by hidden. Cygwin is a windows application so obviously it can get to them as could any other windows application which tried hard enough.

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-04 Thread Christopher Faylor
On Tue, Jul 03, 2007 at 08:42:02AM -0400, Larry Hall (Cygwin) wrote: > On 07/03/2007, Pavel Kudrna wrote: >> > >> > >> > * /From/: "Dave Korn" >> > >> > They are legal, but there's no possible way to convert them to POSIX, >> which >> > has no notion of a per-drive current directory. >> >

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-04 Thread Pavel Kudrna
Larry Hall (Cygwin) wrote: On 07/03/2007, Pavel Kudrna wrote: > > > * /From/: "Dave Korn" > > They are legal, but there's no possible way to convert them to POSIX, which > has no notion of a per-drive current directory. > Ok, but Cygwin != POSIX :-) Cygwin application has environment

Re: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-03 Thread Larry Hall (Cygwin)
On 07/03/2007, Pavel Kudrna wrote: > > > * /From/: "Dave Korn" > > They are legal, but there's no possible way to convert them to POSIX, which > has no notion of a per-drive current directory. > Ok, but Cygwin != POSIX :-) Cygwin application has environment variable for each drive le

RE: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-03 Thread Pavel Kudrna
* /From/: "Dave Korn" They are legal, but there's no possible way to convert them to POSIX, which has no notion of a per-drive current directory. Ok, but Cygwin != POSIX :-) Cygwin application has environment variable for each drive letter to remember current drive. See example and r

RE: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-02 Thread Jörg Schaible
Dave Korn wrote on Monday, July 02, 2007 7:21 PM: > On 02 July 2007 18:17, Pavel Kudrna wrote: > >> Hi, >> the legal win32 paths containing period like "c:.\" or "c:." are >> incorrectly converted by cygwin_conv_to_posix_path() to "c:./" and >> "c:." respectively. See last two output lines of the

RE: bug in cygwin_conv_to_posix_path() caused by period in win32 path

2007-07-02 Thread Dave Korn
On 02 July 2007 18:17, Pavel Kudrna wrote: > Hi, > the legal win32 paths containing period like "c:.\" or "c:." are > incorrectly converted by > cygwin_conv_to_posix_path() to "c:./" and "c:." respectively. See last > two output > lines of the attached example program. > Pavel Kudrna > c:.\c: