bug-findutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Patch] locate --non-existing (-E): New option.


From: James Youngman
Subject: Re: [Patch] locate --non-existing (-E): New option.
Date: Mon, 18 Apr 2005 10:05:02 +0100
User-agent: Mutt/1.3.28i

On Mon, Apr 18, 2005 at 04:44:50AM +0200, Bas van Gompel wrote:

> :  Here is a patch which implements a ``--non-existing'' (``-E'') option
> :  on locate. It's effect is the inverse of ``--existing'' (``-e'').
> 
> :  Along the way I renamed visit_exists_[no]follow to
> :  visit_existing_[no]follow for consistency, and fixed a a few typos in
> :  the description of --nofollow (``treat broken symbolic links /count/
> :  as if they were exi/s/ting'').

I've applied your patch.  Mnay thanks for your contribution.  I did
change a few things though :-

1. Arguments to Texinfo commands are enclosed in {} not (), and so a change 
   to find.texi was required in order for "make" to succeed.

2. I added test suite tests for the -E option (and the -e option).

3. I changed the type of check_existence from into to enum and added
    some comments to locate.c.

I've attached the modified patch, which has already been checked into
CVS.

Regards and thanks again,
James.

Attachment: locate-E.actual.3.patch
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]