Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-03-08 Thread Michael Haggerty
On 02/25/2013 11:50 AM, Michael Haggerty wrote: > On 02/25/2013 10:54 AM, Matthieu Moy wrote: >> [...] Works for me. One minor knit: you've included 10-characters sha1s (this >> comes from >> >> self.short = read_output(['git', 'rev-parse', '--short=10', sha1]) >> >> ), I'd find it better w

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-03-04 Thread Matthieu Moy
Matthieu Moy writes: > A few more random thoughts (not on my personal todo-list): One more: When sending commit emails, it may help to ensure that the dates are strictly monotonic, so that the thread is seen in the right order. IIRC, "git send-email" does this by tweaking the Date: field to ma

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-25 Thread Michael Haggerty
On 02/25/2013 10:54 AM, Matthieu Moy wrote: > Michael Haggerty writes: > >> On 02/20/2013 01:28 PM, Matthieu Moy wrote: >>> Michael Haggerty writes: A while ago, I submitted an RFC for adding a new email notification script to "contrib" [...] >>> >>> We've discussed offline with Michae

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-25 Thread Matthieu Moy
Michael Haggerty writes: > I wonder whether it would be to far off the beaten path to allow glob > patterns in the branch specification; e.g., > >[multimailhook "refs/heads/release-*"] > mailingList = q...@example.com Yes, that would be even better. > For the case of multiple glob

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-25 Thread Matthieu Moy
Michael Haggerty writes: > On 02/20/2013 01:28 PM, Matthieu Moy wrote: >> Michael Haggerty writes: >>> A while ago, I submitted an RFC for adding a new email notification >>> script to "contrib" [...] >> >> We've discussed offline with Michael, a few patches have been merged, >> and there are s

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-23 Thread Michael Haggerty
On 02/20/2013 01:28 PM, Matthieu Moy wrote: > Michael Haggerty writes: >> A while ago, I submitted an RFC for adding a new email notification >> script to "contrib" [...] > > We've discussed offline with Michael, a few patches have been merged, > and there are still a few pending pull requests. I

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-23 Thread Michael Haggerty
On 02/20/2013 01:28 PM, Matthieu Moy wrote: > Michael Haggerty writes: >> A while ago, I submitted an RFC for adding a new email notification >> script to "contrib" [...] > > We've discussed offline with Michael, a few patches have been merged, > and there are still a few pending pull requests. I

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-20 Thread Matthieu Moy
Michael Haggerty writes: > A while ago, I submitted an RFC for adding a new email notification > script to "contrib" [1]. We've discussed offline with Michael, a few patches have been merged, and there are still a few pending pull requests. I liked the script already, but it's getting even cool

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-14 Thread Michael Haggerty
On 02/14/2013 01:55 PM, Matthieu Moy wrote: > Michael Haggerty writes: > >> On 02/13/2013 03:56 PM, Matthieu Moy wrote: >> >>> Installation troubles: >>> >>> I had an old python installation (Red Hat package, and I'm not root), >>> that did not include the email.utils package, so I couldn't use m

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-14 Thread Matthieu Moy
Michael Haggerty writes: > On 02/13/2013 03:56 PM, Matthieu Moy wrote: > >> Installation troubles: >> >> I had an old python installation (Red Hat package, and I'm not root), >> that did not include the email.utils package, so I couldn't use my >> system's python. I found no indication about pyt

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-13 Thread Michael Haggerty
On 02/13/2013 03:56 PM, Matthieu Moy wrote: > Michael Haggerty writes: > >> A while ago, I submitted an RFC for adding a new email notification >> script to "contrib" [1]. The reaction seemed favorable and it was >> suggested that the new script should replace post-receive-email rather >> than b

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-13 Thread Matthieu Moy
Andy Parkins writes: > On Wednesday 13 February 2013 14:56:25 Matthieu Moy wrote: >> Michael Haggerty writes: > >> I think adding a short "dependencies" section in the README (or in an >> INSTALL file) saying which Python version works could save new users the >> trouble (I see the sheebang insi

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-13 Thread Andy Parkins
On Wednesday 13 February 2013 14:56:25 Matthieu Moy wrote: > Michael Haggerty writes: > I think adding a short "dependencies" section in the README (or in an > INSTALL file) saying which Python version works could save new users the > trouble (I see the sheebang inside the scripts says python2 bu

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-02-13 Thread Matthieu Moy
Michael Haggerty writes: > A while ago, I submitted an RFC for adding a new email notification > script to "contrib" [1]. The reaction seemed favorable and it was > suggested that the new script should replace post-receive-email rather > than be added separately, ideally with some kind of migrat

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-01-29 Thread Chris Hiestand
On Jan 29, 2013, at 7:25 AM, Ævar Arnfjörð Bjarmason wrote: > On Sun, Jan 27, 2013 at 9:37 AM, Michael Haggerty > wrote: >> A while ago, I submitted an RFC for adding a new email notification >> script to "contrib" [1]. The reaction seemed favorable and it was >> suggested that the new script

Re: [RFC v2] git-multimail: a replacement for post-receive-email

2013-01-29 Thread Ævar Arnfjörð Bjarmason
On Sun, Jan 27, 2013 at 9:37 AM, Michael Haggerty wrote: > A while ago, I submitted an RFC for adding a new email notification > script to "contrib" [1]. The reaction seemed favorable and it was > suggested that the new script should replace post-receive-email rather > than be added separately, i