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 12:52:34 -0700	[thread overview]
Message-ID: <xmqqikltaghp.fsf@gitster.g> (raw)
In-Reply-To: <PN3PR01MB9597D5EBF0A8D91737E4FE52B89EA@PN3PR01MB9597.INDPRD01.PROD.OUTLOOK.COM> (Aditya Garg's message of "Thu, 22 May 2025 00:53:57 +0530")

Aditya Garg <gargaditya08@live.com> writes:

> I was wondering if it would be acceptable for the maintainers to add a git imap-get-recipients
> command.
>
> I currently am working on it, and it would be a perl script. It would do a very simple thing,
> take the message id as an input, and output the To: and Cc: recipients of that message ID.

If you are selling this tool, you should clarify what the sources
are for the information.  There has to be a database of some sort
that you can query with a message-ID and get addresses in that
message.  What are you using as that database (e.g., your personal
mailbox?  lore archive?  an imap mailbox at your provider?) and how
extensive and configurable is the data source?  What data are you
picking up from that database to come up with To/Cc addresses?

> This can be useful to be used alongwith git-send-email, when you send a v2 and you don't have to
> type all the sender mails again.

FWIW, if you're only duplicating the To/Cc list of the previous
round, then I do not need it, and I do not want to see anybody,
including you, to be using it.  To come up with a list of To/Cc
addresses to use in v2, you should start from those who commented on
v1, in addition to To/Cc used in v1, and then whittle it down.

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.

A more generic version of the response follows to outline the
general principle for those who are watching from sidelines.

----------------------------------------------------------------
[make us come to you, begging]

I've seen from time to time people ask "I am thinking of doing this;
will a patch be accepted?  If so, I'll work on it." before showing
any work, and my response always has been:

 (1) We don't know how useful and interesting your contribution would
     be for our audience, until we see it; and

 (2) If you truly believe in your work (find it useful, find writing
     it fun, etc.), that would be incentive enough for you to work
     on it, whether or not the result will land in my tree.  You
     should instead aim for something so brilliant that we would
     come to you begging for your permission to include it in our
     project.


  reply	other threads:[~2025-05-21 19: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 [this message]
2025-05-22  1:52   ` Junio C Hamano
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=xmqqikltaghp.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).