RFC, patch -- loginfo behavior for commitinfo

[Logan Johnson]

Hello all,

My company has implemented a commit control system, but it is
currently applied by a call from loginfo, which means that the best
we can do is sound an alarm if a commit we don't like goes through.
We need it at loginfo so that we can take into account the log
message, the branch, and the operation on each file involved.  

In order to make this system more effective, and eliminate some
internal spamming, I've hacked cvs-1.11.1p1 so that the log message
is obtained *before* commit, the loginfo filter handler
(logfile_write()) returns the status of the pipe when it closes, and
commitinfo's lines are processed by logfile_write().  

What I'd like to know is whether this is a horrible idea (I realize
it breaks compatibility with current commitinfo behavior, but we've
decided not to care).  More importantly, because I'm reasonably sure
this is what we want to do, I'd like to know if my implementation
(read: "hack")-- attached-- seems likely in anyone's mind to have 
unintended consequences.  I'll be testing it here, of course, but I 
have limited experience with CVS.  Any comments would be greatly
appreciated.


-- 
   Logan Johnson     :: address@hidden
   Terraspring, Inc. :: http://www.terraspring.com

cvs-1.11.1p1-log_to_commitinfo.diff
Description: cvs-1.11.1p1-log_to_commitinfo.diff