git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Aditya Garg <gargaditya08@live.com>
Cc: "git@vger.kernel.org" <git@vger.kernel.org>
Subject: Re: Add git imap-get-recipients command
Date: Wed, 21 May 2025 18:52:10 -0700	[thread overview]
Message-ID: <xmqqfrgx76ph.fsf@gitster.g> (raw)
In-Reply-To: <xmqqikltaghp.fsf@gitster.g> (Junio C. Hamano's message of "Wed, 21 May 2025 12:52:34 -0700")

Junio C Hamano <gitster@pobox.com> writes:

> Again, the description of the "tool" in the first paragraph was so
> sketchy that I cannot tell where you are gathering the To/Cc
> addresses from or if the tool is using only the named message, or
> considers messages sent as response to that named message, so it is
> impossible to give a meaningful response.  We cannot tell if the
> tool will be useful with given information.

I almost forgot to mention a few more things:

 - Perl dependency is something we seem to be avoiding these days.

 - Adding new things to contrib/ is also something we seem to be
   avoiding.

 - If it is only to help people to contribute to this project, I am
   not sure where in the hierarchy in my tree the tool should go.
   If it is to help people to contribute to any Git managed project,
   then it can be thrown into the same category as git-send-email,
   though.

 - Doesn't contributor side of b4 [*] do more or less what you want?
   If you find it lacking, it still may be a better idea to extend
   it instead of starting from scratch.


[Reference]

 * https://b4.docs.kernel.org/en/latest/contributor/overview.html

   Admittedly I do not use the contributor side of the tool,
   although I often use it for maintainer's tasks.  b4 knows the
   threading, what set of messages in the thread represents the
   latest iteration, etc., so it should already know enough to find
   out things like "who reviewed the previous round" that would help
   solving your "whom should I Cc this new iteration of patches?"
   problem.

  reply	other threads:[~2025-05-22  1:52 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-21 19:23 Add git imap-get-recipients command Aditya Garg
2025-05-21 19:52 ` Junio C Hamano
2025-05-22  1:52   ` Junio C Hamano [this message]
2025-05-22  3:20   ` Aditya Garg
2025-05-22 17:54     ` Emily Shaffer
2025-05-22 17:59       ` Aditya Garg
2025-05-22 18:55       ` Junio C Hamano

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=xmqqfrgx76ph.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=gargaditya08@live.com \
    --cc=git@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).