Re: [PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-22 Thread Junio C Hamano
Junio C Hamano writes: >> I think the notice still deserve the "***NOTICE***" or whatever makes it >> visible enough to distinguish it from the traditional licence & >> non-warranty header, but I don't think we should kill the old script too >> early. > > True. I personally felt that Jonathan's

Re: [PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-15 Thread Junio C Hamano
Matthieu Moy writes: > Jonathan Nieder writes: > >> (3) >> # An example hook ... >> # >> # Warning: this script is no longer actively maintained. Consider >> # switching to ... >> >> I prefer (2), which makes it clear to the reader that it is dangerous >> to keep using the s

Re: [PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-15 Thread Matthieu Moy
Jonathan Nieder writes: > (3) > # An example hook ... > # > # Warning: this script is no longer actively maintained. Consider > # switching to ... > > I prefer (2), which makes it clear to the reader that it is dangerous > to keep using the script (since no one is activel

Re: [PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-15 Thread Michael Haggerty
On 07/15/2013 08:02 AM, Jonathan Nieder wrote: > Michael Haggerty wrote: > >> Add a notice to the top of post-receive-email explaining that the >> script is no longer under active development and pointing the user to >> git-multimail. > > I think the spirit of this patch is sane. Some thoughts o

Re: [PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-14 Thread Jonathan Nieder
Michael Haggerty wrote: > Add a notice to the top of post-receive-email explaining that the > script is no longer under active development and pointing the user to > git-multimail. I think the spirit of this patch is sane. Some thoughts on wording: [...] > --- a/contrib/hooks/post-receive-email

[PATCH v4 2/2] post-receive-email: deprecate script in favor of git-multimail

2013-07-14 Thread Michael Haggerty
Add a notice to the top of post-receive-email explaining that the script is no longer under active development and pointing the user to git-multimail. Signed-off-by: Michael Haggerty --- If you think this is premature or the wording is too strong; feel free to change or omit this patch. I think