Re: [ANNOUNCE] cvsps support for parsing BK->CVS kernel tree logs

[Andrea Arcangeli]

On Wed, Mar 19, 2003 at 05:21:12PM -0500, David Mansfield wrote:
> On Wed, 19 Mar 2003, Andrea Arcangeli wrote:
> 
> > I'm downloading the new version now... ;) thanks
> > 
> > > The file is actualy a substring match.  If the -f argument matches as a 
> > 
> > so it doesn't sound a regex. Being able to specify more than 1 -f would
> > be very useful. Either that or regex would do it fine too with
> > '^net/core', so as far as I can write stuff like -f
> > '^net/core|^net/ipv4' I'm fine.
> > 
> > I also think using match by default in the regex is cleaner. So I can
> > write -f 'net/core|net/ipv4' w/o bothering about the ^ because it become
> > implicit. And I can still use '.*net/core.*' if I want a substring
> > regex. I think substring search will be not common.
> > 
> > But really, any kind of way you implement the 'multiple file' thing is
> > fine as far as I can specify more than 1 file ;).
> > 
> 
> Attached is a patch (on top of previous which is on top of 2.0b5) changes 
> the -f to regex match.  The regex is of course slightly slower than 
> 'strstr' but I agree the advantage is worth it.
> 
> It differs slightly in syntax (c library regex just works this way):
> 
> cvsps -f '^net/core\|^net/ipv4'
>                    ^
>                    You must escape the | symbol to separate the regex.
> 
> Try it out!

ouch I did it too ;) I used the EXP_EXTENDED so I don't need to use \|,
I'm not used to it.

Andrea