This is the mail archive of the
cygwin-patches@cygwin.com
mailing list for the Cygwin project.
Re: [Patch]: mkdir -p and network drives
- From: Christopher Faylor <cgf-no-personal-reply-please at cygwin dot com>
- To: cygwin-patches at cygwin dot com
- Date: Fri, 6 May 2005 10:22:13 -0400
- Subject: Re: [Patch]: mkdir -p and network drives
- References: <002f01c5523f$6d6f38b0$3e0010ac@wirelessworld.airvananet.com>
- Reply-to: cygwin-patches at cygwin dot com
On Fri, May 06, 2005 at 09:27:55AM -0400, Pierre A. Humblet wrote:
>cgf wrote:
>> On Thu, May 05, 2005 at 10:57:08PM -0400, Pierre A. Humblet wrote:
>
>>>The code should handle "//" correctly, but path.cc still transforms it
>>>into "/", because of the bash bug.
>
>> Is that fixed in the current bash?
>
>AFAIK Corinna fixed it once, but the patch got lost and it's currently
>not fixed.
>
>> So, I'd appreciate it if you would just move your fhandler_netdrive
>> stuff to fhandler_netdrive.cc.
>
>Sure. Thanks for setting up the framework.
>
>> I didn't renumber FH_FS with above change. I wasn't sure why you did
>> that. I don't think that there was a requirement that it has to be the
>> lowest numbered minor device number. If there is a requirement like
>> that we should change it.
>
>OK. No requirement, just aesthetic. There seemed to be a pattern.
>
>>>About implementing readdir: PTC...
>
>> I was thinking about doing this but how would it ever be invoked?
>
>With "ls -l //" or "ls -l //machine"
>
>> You can't do an opendir on "//", right?
>
>Sure you can (thanks to existing code in the virtual driver). Just remove
>the
>code in path.cc that changes "//" into "/". It's only/mainly there because
>of bash.
Well, that was kinda my point. If we can't remove the "//" handling because
it breaks bash then adding opendir/readdir stuff seems premature except for
the case of ls //foo which is entirely different from ls //.
cgf