[Kde-scm-interest] Post-commit hooks script
Oswald Buddenhagen
ossi at kde.org
Sat Dec 5 00:11:19 CET 2009
On Fri, Dec 04, 2009 at 07:49:52PM +0100, Jean-Nicolas Artaud wrote:
> # One mail send for each CCMAIL
> for myMail in email_addresses
> mailIo = IO.popen("/usr/sbin/sendmail #{myMail}", 'w')
>
that's a bad idea. for one, each mesage has a separate message id, which
makes duplicate elimination not work. second, spawning a separate
sendmail for each mail is wasting resources.
as you are ignoring refs other than the currently updated one, you'll
re-send all mails when a branch from the same repo is merged.
you are entirely ignoring the existence of new refs, which will lead to
kinda interesting effects when someone creates a bugfix branch.
quite frankly, you did the trivial work. you can either do the hard
stuff and spend a lot of time on it or just wait for me to finish -
there are enough other things to do, and i'll do my script anyway, as i
a) need it for another project and b) want something generic for
submission to git upstream.
More information about the Kde-scm-interest
mailing list