info-cvs
[Top][All Lists]
Advanced

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

Re: ANN: CVSToys 1.0.0 - commit notification extravaganza


From: Kevin Turner
Subject: Re: ANN: CVSToys 1.0.0 - commit notification extravaganza
Date: 11 Nov 2002 00:58:54 -0800

On Mon, 2002-11-11 at 00:13, Greg A. Woods wrote:
> > Unlike 'loginfo' alone, which will only execute the first
> > matching command it finds, FreshCVS's flexible configuration allows for
> > any number of actions to take place.
> 
> That's bogus.  CVS has always had an "ALL" keyword.

Which gives you ALL or nothing.  That doesn't handle something like

X* command1
Xsub* command2

Y* command3

...you don't want ALL to happen for any of commands 1, 2, or 3, but you
do want both 1 and 2 to happen for Xsubfoo.

> > Another key feature of FreshCVS notifications is that you receive
> > one per commit, not one for each directory of the commit as
> > loginfo-triggered scripts are prone to do.
> 
> That's also more or less bogus.  There's been comit_prep and
> log_accum in the contrib directory for nearly forever and they do
> exactly the same thing.

Acknowledged.  And in fact, I took a page from log_accum's book (hooking
commitinfo) when I wanted to get rid of my Linux-specific tricks.  But
log_accum does one thing; perhaps not the thing I wanted to do.  The
idea was to factor that capability out instead of reproducing it in
every script.  I've only just today seen a version of log_accum with
some concept of plug-ins[1], which is more along the lines of what I
wanted.

> > FreshCVS is extensible with Python, and you may also publish
> > notifications by a Perspective Broker network service, allowing 3rd
> > party clients to listen in.
> 
> Now that's sort of interesting.  (Scheme would have been better :-)

There's a Perspective Broker implementation for elisp, is that close
enough?  ;)

Thanks for the feedback,

 - Kevin


[1] That's the version of log_accum that's in 
http://cvs.sourceforge.net/cgi-bin/viewcvs.cgi/cvsql/cvs-scripts/log_accum?rev=HEAD&content-type=text/vnd.viewcvs-markup






reply via email to

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