#40 open
Jeremy Lightsmith

[PATCH] Support for post-commit hook

Reported by Jeremy Lightsmith | April 21st, 2008 @ 01:47 PM

For various reasons, the people who host our Subversion repository would prefer that we don't poll the repository. So I wrote the attached patch to support a post-commit hook. The changes add a new notification controller, which supports a single dirs_changed action. This works together with a simple post-commit hook which sends a list of changed directories. Here's an example post-commit hook: ------------------------ #!/bin/sh WGET=/opt/local/bin/wget SVNLOOK=/opt/local/bin/svnlook CC_URL=http://cruise.82ask.com/notification/dirs_changed $WGET -q -O- --post-data="`$SVNLOOK dirs-changed $1`" $CC_URL > /dev/null ------------------------ I think that I've made these changes in a "sympathetic" way to the existing code, but have had to restructure the PollingScheduler code somewhat. At some point, I should probably modify PollingController so that it can be configured so that it doesn't poll at all (we currently run with the polling interval set to 12.hours). Please let me know what you think of these changes - if there are things that should be changed in order to make this fit in with the "CruiseControl way", I'd love to hear about them.

Comments and changes to this ticket

  • cosjav

    cosjav August 18th, 2008 @ 09:55 PM

    • Tag set to new feature, major

    Has anything happened since this patch?

    By the way I cant find the attachment that you are referring to. I understand the post-commit hook script but what other code changes did you make for this patch?

    I'd be interested in knowing whether this is being planned as an inbuilt feature of cruise control seeing as most versioning control applications have 'hook' like functionality by now.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

People watching this ticket

Pages