git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] contrib/hooks/post-receive-email: get description from repo.git/config

From
Michael Haggerty <mhagger@alum.mit.edu>
Date
Jul 2, 2013, 09:12 UTC
Message-ID
<51D2996B.8070503@alum.mit.edu>
In-Reply-To
<7v1u7iapzw.fsf@alter.siamese.dyndns.org>
On 07/01/2013 11:58 PM, Junio C Hamano wrote:
Show 19 quoted lines
> Michael Haggerty <mhagger@alum.mit.edu> writes:
> 
>> My understanding is that we are waiting on two things:
>>
>> 1. Consensus from the community.  I would characterize the feedback on
>> the mailing list as limited in quantity but strongly positive [1-4] and
>> I think that most/all of the wishes for post-receive-email features that
>> were originally omitted from git-multimail have been implemented in the
>> current version.  Some of the mailing list feedback was about earlier
>> versions.  Do you want people to give feedback specifically about the
>> current version?
>>
>> 2. For me to figure out what part of the git-multimail history I think
>> should be included in the Git project, do any necessary repository
>> rewriting, and submit a pull request to you.  The fact that I haven't
>> gotten to this is due to the fact that I've been busy getting git-imerge
>> [5] ready to present at GitMerge.
> 
> Ping, now GitMerge is over?
Yes, and its reverberations are slowly getting under control, too :-)
> No need to hurry, but just to make sure this didn't disappear from
> everybody's radar.

I definitely haven't forgotten it. I wasn't so happy with the script's Python API, and was holding off on the final submission to avoid casting the old way in stone. But I finally had some time over the last two weekends to convert to what I think is a more sensible system [1]. (I also improved the test coverage considerably.)

Feedback would be very welcome, especially from people who have tried out the old Python API. I am even willing to personally help people adapt to the new API, as it would help me verify that real-life customizations are now indeed easier. (Ævar, any news?)

This week I want to convert $WORK to the new version; after that I would feel comfortable submitting to Git contrib.

(By the way, the changes don't affect how the script can be configured via "git config" settings, or its backwards compatibility with post-receive-email. The improvements are all to the Python API.)

I have a logistical question: git-multimail doesn't have its own mailing list, and GitHub doesn't offer one. I was thinking about setting up a Google group, but a few people at GitMerge suggested that I instead direct discussion of git-multimail to the main Git mailing list. I would slightly prefer that, but I would first like to make sure that the extra traffic (probably not much) would be welcome on the Git mailing list.

Michael
[1] "master" branch at https://github.com/mhagger/git-multimail
-- 
Michael Haggerty
mhagger@alum.mit.edu
http://softwareswirl.blogspot.com/
Previous: Junio C HamanoNext: Junio C Hamano
Message 11 of 15 in “contrib/hooks/post-receive-email: get description from repo.git/config”
  1. contrib/hooks/post-receive-email: get description from repo.git/configTrond Hasle Amundsen, May 6, 2013
  2. Trond Hasle AmundsenMay 6, 2013
  3. Junio C HamanoMay 6, 2013
  4. Matthieu MoyMay 6, 2013
  5. Trond Hasle AmundsenMay 6, 2013
  6. Junio C HamanoMay 7, 2013
  7. Michael HaggertyMay 7, 2013
  8. Junio C HamanoMay 7, 2013
  9. Matthieu MoyMay 7, 2013
  10. Junio C HamanoJul 1, 2013
  11. Michael HaggertyJul 2, 2013
  12. Junio C HamanoJul 2, 2013
  13. Matthieu MoyMay 7, 2013
  14. Junio C HamanoMay 7, 2013
  15. Trond Hasle AmundsenMay 6, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.