From: Michael Haggerty <mhagger@alum.mit.edu>
To: Junio C Hamano <gitster@pobox.com>
Cc: "Matthieu Moy" <Matthieu.Moy@grenoble-inp.fr>,
"Trond Hasle Amundsen" <t.h.amundsen@usit.uio.no>,
git@vger.kernel.org, "Marc Branchaud" <mbranchaud@xiplink.com>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
"Chris Hiestand" <chiestand@salk.edu>,
"Jeff King" <peff@peff.net>
Subject: Re: [PATCH] contrib/hooks/post-receive-email: get description from repo.git/config
Date: Tue, 02 Jul 2013 11:12:11 +0200 [thread overview]
Message-ID: <51D2996B.8070503@alum.mit.edu> (raw)
In-Reply-To: <7v1u7iapzw.fsf@alter.siamese.dyndns.org>
On 07/01/2013 11:58 PM, Junio C Hamano wrote:
> 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/
next prev parent reply other threads:[~2013-07-02 9:12 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-06 13:53 [PATCH] contrib/hooks/post-receive-email: get description from repo.git/config Trond Hasle Amundsen
2013-05-06 15:22 ` Trond Hasle Amundsen
2013-05-06 15:36 ` Junio C Hamano
2013-05-06 15:42 ` Matthieu Moy
2013-05-06 16:29 ` Trond Hasle Amundsen
2013-05-07 6:36 ` Junio C Hamano
2013-05-07 7:46 ` Michael Haggerty
2013-05-07 15:15 ` Junio C Hamano
2013-05-07 15:36 ` Matthieu Moy
2013-07-01 21:58 ` Junio C Hamano
2013-07-02 9:12 ` Michael Haggerty [this message]
2013-07-02 18:51 ` Junio C Hamano
2013-05-07 7:49 ` Matthieu Moy
2013-05-07 14:51 ` Junio C Hamano
2013-05-06 16:26 ` Trond Hasle Amundsen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51D2996B.8070503@alum.mit.edu \
--to=mhagger@alum.mit.edu \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=avarab@gmail.com \
--cc=chiestand@salk.edu \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=mbranchaud@xiplink.com \
--cc=peff@peff.net \
--cc=t.h.amundsen@usit.uio.no \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.