[cvsspam-devel] Patches for Gforge
Raphaël Pinson
raphink at gmail.com
Fri Aug 21 16:59:08 BST 2009
2009/8/21 Raphaël Pinson <raphink at gmail.com>:
> Hello,
>
> We have just set up CVSSpam for a Gforge platform, and I'm attaching
> two patches and one file to this email.
>
> * collect_diffs_fix_root_commits.diff is a small fix. I happen to have
> some files that are not in modules but directly in the root of the CVS
> module. This patch allows to send diffs for modifications of such
> files ;
> * gforge_scm.diff adds support for the SCM plugin for Gforge for
> cvsspam.rb. In order to use it, you need to add the following two
> parameters to your cvsspam.conf :
> $gforgescmURL = "http://gforge.example.org/scm"
> $repository_name = GUESS
> * cvsspam-wrapper.sh is a little script which makes it easier to call
> cvsspam on Gforge. It wraps the call to collect_diffs.rb by using the
> noreply@ account as from address by default and
> ${repository}-commits at lists as to address by default. I've added
> Roland to the recipients of this mail in case he is interested by
> this.
>
I'm sorry to say there were two bugs in one of my patches and I only
found it now (my ruby skills are very bad).
So here is the fixed version of gforge_scm.diff which fixes :
* unnecessary use of initalize provoked warning ;
* not using a tag in a commit crashed cvsspam, so now I check for tag==nil.
Regards,
Raphaël Pinson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gforge_scm.diff
Type: text/x-diff
Size: 1629 bytes
Desc: not available
URL: <http://lists.badgers-in-foil.co.uk/pipermail/cvsspam-devel/attachments/20090821/4ca364b1/attachment.diff>
More information about the cvsspam-devel
mailing list