Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Date: Sun, 12 May 2002 15:23:04 -0400
From: Christopher Faylor <cgf-cygwin AT cygwin DOT com>
To: cygwin-patches AT cygwin DOT com, cygwin AT cygwin DOT com
Subject: Re: [PATCH] Get recursive grep to work on Win9x
Message-ID: <20020512192304.GA2518@redhat.com>
Reply-To: cygwin-patches AT cygwin DOT com, cygwin AT cygwin DOT com
Mail-Followup-To: cygwin-patches AT cygwin DOT com, cygwin AT cygwin DOT com
References: <3CDE8B30 DOT 95496DFA AT cistron DOT nl>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <3CDE8B30.95496DFA@cistron.nl>
User-Agent: Mutt/1.3.23.1i

On Sun, May 12, 2002 at 05:33:04PM +0200, Ton van Overbeek wrote:
>The patch is simple. It makes use of the wincap.can_open_directories()
>capability, which seems to be foreseen for exactly this type of problems.
>However I could not find an other place where this capability is used
>in the cygwin.dll.
>
>Patch attached: fhandler.diff

Thanks for the patch.

I've checked in a slightly different version which should have the same
effect.

I appreciate the time you spent tracking this down and researching the
correct method for fixing this.  I used roughly the same technique as
you but moved it to only affect the opening of disk files.  Hopefully,
it will fix your problem.

Btw, for future reference, your patch was reversed.  It should have been
diff -u foo.orig foo, i.e. the original file goes first.

Thanks again for tracking this down.

cgf

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Bug reporting:         http://cygwin.com/bugs.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/