From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7530 invoked by alias); 7 Feb 2013 16:24:10 -0000 Received: (qmail 7212 invoked by uid 22791); 7 Feb 2013 16:23:32 -0000 X-Spam-Check-By: sourceware.org Received: from aquarius.hirmke.de (HELO calimero.vinschen.de) (217.91.18.234) by sourceware.org (qpsmtpd/0.83/v0.83-20-g38e4449) with ESMTP; Thu, 07 Feb 2013 16:23:25 +0000 Received: by calimero.vinschen.de (Postfix, from userid 500) id 35374520351; Thu, 7 Feb 2013 17:23:22 +0100 (CET) Date: Thu, 07 Feb 2013 16:24:00 -0000 From: Corinna Vinschen To: cygwin@cygwin.com Subject: Re: stat() and tilde prefix (was bad bash tab completion) Message-ID: <20130207162322.GB15210@calimero.vinschen.de> Reply-To: cygwin@cygwin.com Mail-Followup-To: cygwin@cygwin.com References: <5024B4D4.6080409@shaddybaddah.name> <50F395D5.4050201@shaddybaddah.name> <20130114061747.GB16739@ednor.casa.cgf.cx> <20130114100002.GA22039@calimero.vinschen.de> <50F417F9.8040108@shaddybaddah.name> <20130114161307.GB8617@calimero.vinschen.de> <50F54C93.9090702@shaddybaddah.name> <511350F3.8040702@shaddybaddah.name> <5113C886.9070904@redhat.com> <5113D1ED.20305@towo.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <5113D1ED.20305@towo.net> User-Agent: Mutt/1.5.21 (2010-09-15) Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@cygwin.com Mail-Followup-To: cygwin@cygwin.com X-SW-Source: 2013-02/txt/msg00074.txt.bz2 On Feb 7 17:10, Thomas Wolff wrote: > Am 07.02.2013 16:30, schrieb Eric Blake: > >... > > > >... > >the fact that cygwin's handling of .. is not POSIX-compliant. I think a > >better fix would be to change file_exists() itself instead of adding a > >misnamed wrapper function; then bashline.c wouldn't even need patching. > > The string 'tilde' need not even be in the patch; what you are really > >after is a function that says that if '..' is found within a string > >being probed for existence, then add an additional check to see if the > >prefix of that string exists as a directory. > > > >But I don't mind experimenting with the idea - it remains to be seen > >whether people will complain that bash is noticeably slower because it > >takes time to double-check instead of rely on cygwin's non-POSIX > >shortcut. And the slowdown would only be on paths containing a '..'; I > >would NOT be checking for symlinks (even though symlinks containing .. > >are also being interpreted in a non-POSIX manner, it is much more > >expensive to second-guess if you have to check every name for being a > >symlink than it is to just check for literal ..). > Do I interpret correctly that you talk about bash filename completion here? > Referring to http://cygwin.com/ml/cygwin/2013-01/msg00201.html, > I'd like to point out that while this ".." thing is a cygwin bug, or > known downside as Corinna says, > the same issue occurs on Linux precisely with filename completion > which isn't consistent there either. > So it would be "over-fixing" to handle that specifically in bash. > > On the other hand, considering again this "downside": > If the core cygwin filesystem function would follow this approach, > simply checking for an occurrence of ".." first before resolving the > filename, > wouldn't that be an acceptable fix without inappropriate penalty? You don't know what you're asking for (can of worms, etc.) This is some kind of chicken egg problem. - The path must be normalized, otherwise we can't reliably convert the POSIX path prefix to a Windows pathname. - Only after converting to a Windows path, we can perform file checks. - Rinse and repeat. Also, the path normalization is performed in an entirely distinct function from the mount point conversion, and this in turn is another function than the path function handling symlinks and devices. Changing that requires to implement the path conversion functionality almost from scratch. Given the age of some of these functions, I'd like to have done that for a long time, but I'm constantly shying away since I don't want to break what is working today. There's, of course, still the aforementioned chicken-egg problem. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat -- 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