Date: Sat, 1 Dec 2001 23:22:16 -0500 From: Garance A Drosihn <drosih@rpi.edu> To: Mike Barcroft <mike@FreeBSD.org>, Josef Karthauser <joe@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: CVSROOT cfg.pm log_accum.pl Message-ID: <p05101005b82f5a88965a@[128.113.24.47]> In-Reply-To: <20011201214307.A46242@espresso.q9media.com> References: <200112011138.fB1BciW72956@freefall.freebsd.org> <20011201214307.A46242@espresso.q9media.com>
next in thread | previous in thread | raw e-mail | index | archive | help
At 9:43 PM -0500 12/1/01, Mike Barcroft wrote: >Josef Karthauser <joe@FreeBSD.org> writes: > > > > Modified files: >> . cfg.pm log_accum.pl >> Log: >> Add the ability to have diffs inserted into the commit email. >> > > Submitted by: jesper > >cvs-verbose@FreeBSD.org anyone? This would lower the barrier to >entry for reviewing commits. > >Best regards, >Mike Barcroft An alternate possibility: Put the diff's for a given commit up as a web page, and have the URL to that web page be in the regular commit mail. This would be 'diff -u' output for all the files which changed in a commit, which would be more convenient than going thru the CVSweb interface to separately pull up the commit to each file. If we use a separate mailing list, then I won't want to join the mailing list because the vast majority of changes will be ones that I do not care about. But for a few areas (or interesting-sounding changes), it would be nice if I could pull up all the diffs for everything changed in a commit by just clicking on a URL. This is just a vague idea I've had for awhile... I'm not completely sure how to put it together so it would work smoothly. -- Garance Alistair Drosehn = gad@eclipse.acs.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p05101005b82f5a88965a>