From: Junio C Hamano <junkio@cox•net>
To: Andy Parkins <andyparkins@gmail•com>
Cc: git@vger•kernel.org
Subject: Re: [PATCH] update-hook: remove all functionality that should be in hooks/post-receive
Date: Fri, 23 Mar 2007 18:09:56 -0700 [thread overview]
Message-ID: <7v648ro3a3.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <200703231022.00189.andyparkins@gmail.com> (Andy Parkins's message of "Fri, 23 Mar 2007 10:21:59 +0000")
Andy Parkins <andyparkins@gmail•com> writes:
> Now that we have a post-receive hook; the update hook's only job is to
> decide is a particular update is allowed or not.
>
> This example hook removes all of the functionality that should now
> rightly be done by the post-receive hook. In particular: the generation
> of notification emails.
>
> Signed-off-by: Andy Parkins <andyparkins@gmail•com>
I was quite impressed by seeing what Shawn did with his
'continuous' stuff, not the way it is implemented, but the way
it was presented as "you can do this to use it from your hook
script".
You know what? I am very tempted to take this patch, while
dropping the other one. Well, dropping is probably not quite
accurate, because being a nice person (and I am good looking,
too ;-), I would probably end up creating "contrib/mailhook/"
hierarchy and stash the contents of your second patch there
myself.
I think I'd better let fancier hooks live in contrib/examples
hierarchy for people to pick and choose, and keep the default
templates/ directory lean and clean.
There is a small detail of how users who use prepackaged git
would get rich library of example hooks from, but that is
something better left to distro people; that way I do not have
to worry about them too much, and it would also make it crystal
clear that these are just examples.
The thing is, not many people are interested in sending e-mail
out from post- any hooks (I don't do so, Linus doesn't either),
and there is no strong justifiation to give e-mail sending users
any preferential treatment and penalize others by copying rather
huge hook scripts from templates/ that they are not going to
ever use.
next prev parent reply other threads:[~2007-03-24 1:10 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-23 10:21 [PATCH] update-hook: remove all functionality that should be in hooks/post-receive Andy Parkins
2007-03-24 1:09 ` Junio C Hamano [this message]
2007-03-24 3:23 ` Linus Torvalds
2007-03-24 3:28 ` Shawn O. Pearce
2007-03-24 8:14 ` Andy Parkins
2007-03-24 10:26 ` Theodore Tso
2007-03-24 10:41 ` Junio C Hamano
2007-03-24 12:44 ` Andy Parkins
2007-03-24 14:21 ` Theodore Tso
2007-03-24 14:43 ` Francis Moreau
2007-03-24 15:48 ` Andy Parkins
2007-03-24 18:15 ` Francis Moreau
2007-03-25 9:11 ` Andy Parkins
2007-03-25 9:25 ` Shawn O. Pearce
2007-03-25 14:04 ` Francis Moreau
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7v648ro3a3.fsf@assigned-by-dhcp.cox.net \
--to=junkio@cox$(echo .)net \
--cc=andyparkins@gmail$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox