[prev in list] [next in list] [prev in thread] [next in thread] 

List:       git
Subject:    Re: [RFC] Automagic patch merge ack emails
From:       "Boyd Stephen Smith Jr." <bss () iguanasuicide ! net>
Date:       2008-12-22 20:59:26
Message-ID: 200812221459.30728.bss () iguanasuicide ! net
[Download RAW message or body]

On Monday 2008 December 22 08:07:07 Richard Hartmann wrote:
> I poked around the docs, but I could not find any option to have git
> send email to people who signed off a patch when it's merged.

There's not one, since something that that should probably be done in a hook.  
Maybe the update hook?

> I only realized the two patches I sent were merged because they were
> listed in the RC changelog summary and would have followed up my patch
> email in a about a week, asking about its status.
>
> Does anyone else think this is useful?

I think it would be nice to get a notification whenever Junio pushes commits 
to 'master' or 'maint' in git.git if I've signed off on one of the commits.  
I don't think everyone would though so the hook would need to be maintained 
indefinitely (updating the email blacklist, etc.).

It would probably be a nice hook to have as an example, I guess.

> Does anyone else think it should 
> make it into main so it can be enabled via config, not via a hook that
> needs to be imported into each and every repo?

Definitely not.  It's not that difficult to add a hook to the repositories 
that need it, and I think the majority of repositories will be private and 
not need it.
-- 
Boyd Stephen Smith Jr.                     ,= ,-_-. =. 
bss@iguanasuicide.net                     ((_/)o o(\_))
ICQ: 514984 YM/AIM: DaTwinkDaddy           `-'(. .)`-' 
http://iguanasuicide.net/                      \_/     

["signature.asc" (application/pgp-signature)]
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic