Re: CVS commit: src/usr.bin/find

[James Chacon <jmc%NetBSD.org@localhost>]

On Wed, Oct 19, 2005 at 02:23:45PM +0200, Quentin Garnier wrote:
> On Wed, Oct 19, 2005 at 12:17:22PM +0000, Elad Efrat wrote:
> > 
> > Module Name:        src
> > Committed By:       elad
> > Date:               Wed Oct 19 12:17:22 UTC 2005
> > 
> > Modified Files:
> >     src/usr.bin/find: main.c
> > 
> > Log Message:
> > Strip trailing slashes before using the path.
> > PR/31869.
> 
> That's completely bogus.
> 
> find /
> find /somedir/ /someotherdir/
> 
> clearly don't work as expected.  Please back this out.
> 

Yeah, blindly stripping all /'s off until you find something else isn't
the way to go.

(and this code underflows with just "/")

I agree stripping extraneous /'s off makes sense but do it sanely. (and add
a regress test)

James