Re: Thanks for cvs2cl.pl

On Fri, Dec 14, 2001 at 11:44:27AM -0500, Chris Jackson wrote:
> Ahhh - cvs2cl.pl was just what we needed - Thanks!
> I've attached a diff of a tweak we added to allow for regex's that
> exclude lines (--notregex 'regex' or -N 'regex'), like how -R 'regex'
> exclusively includes matching lines.
> Please do what you want with the patch - no credits needed.

Hello Chris,

My name is Martyn Pearce, and I've taken over maintenance of cvs2cl.pl . I
apologize for taking so long to get back to you; I'm sifting through old bug
reports & feature requests now.

I very nearly applied your patch (thanks for the patch --- reports with
patches get priority!) --- but then, I suddenly thought "what's the difference
between exluding a log based on a regex, rather than including only logs that
don't match a regex?". I don't think there is one (a difference, that is).
So, I think that that exclude facility may easily be handled using -R '!regex'
rather than -N 'regex'.

If you have a counterpoint to that, please show me. I'm very happy to be
convinced. Otherwise, I'm inclined to not add another option now for
something that can already be achieved, because cvs2cl(.pl) already has plenty
of options...

Is this okay with you?

Mx.
Received on Mon Nov 18 09:24:23 2002