Subject: Re: [RFC] Add a new email notification script to "contrib"

From: Michael Haggerty <[email-masked]>
Date: Mon, 16 Jul 2012 21:05:07 +0200

On 07/16/2012 07:47 PM, Junio C Hamano wrote:
> Michael Haggerty <[email-masked]> writes:
>
>> It's already supported:
>>
>>      git config hooks.announcelist \
>>          "[email-masked], [email-masked], [email-masked]"
>>      git config hooks.refchangelist \
>>          "[email-masked], [email-masked]"
>>      git config hooks.commitlist "[email-masked]"
>
> Sad.  Is the "post-recieve email" squatting on the _whole_ hooks.*
> namespace, not postreceiveemailhook.* or something more reasonably
> limited?

Yes.  I didn't like this either, but I copied this aspect of the old 
script in the interest of compatibility.

If we want to change this, now would be a good time, before the new 
script starts using more of the hooks namespace.  One could choose a new 
namespace and deprecate the old one, optionally in multiple steps:

1. Only read new options from new namespace; read old options from new 
namespace or (if the new names are not set) falling back to the old 
namespace.

2. Deprecate the old namespace, emitting warnings if the old names are 
still set but continuing to use them.

3. Stop supporting the old namespace, but emit warnings if the old names 
are still set.

4. Stop looking in the old namespace altogether.

Though, given that hook scripts have to be enabled/upgraded 
per-repository, usually on a central server, probably doing only steps 2 
and 4 would be enough.

Michael

-- 
Michael Haggerty
[email-masked]
http://softwareswirl.blogspot.com/