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

Re: Cc'ing the Git maintainer on GitGitGadget contributions, was Re: [PATCH 0/1] add--interactive: skip index refresh in reset patch mode

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Jan 7, 2021, 14:57 UTC
Message-ID
<87wnwordzh.fsf@evledraar.gmail.com>
In-Reply-To
<nycvar.QRO.7.76.6.2101071517260.2213@tvgsbejvaqbjf.bet>
On Thu, Jan 07 2021, Johannes Schindelin wrote:
Show 29 quoted lines
> Hi Junio,
>
> On Tue, 26 Nov 2019, Junio C Hamano wrote:
>
>> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>>
>> >> > Hmph, I wonder why this was sent my way.  How does GGG determine
>> >> > whom to send patches to?  I, like other reviewers, prefer not to see
>> >> > earlier rounds of patches sent directly to me unless they are about
>> >> > areas that I am mostly responsible for (other patches I'll see them
>> >> > and review them on the copies sent to the mailing list anyway).
>> >
>> > Oops, I forgot to address this. The reason why this is sent your way is
>> > that you are the Git maintainer, and as such, GitGitGadget sends _all_ Git
>> > patches your way (except the Git GUI ones).
>> >
>> > The reason for this is that this is the suggested way, as per
>> > https://git-scm.com/docs/SubmittingPatches#patch-flow:
>> >
>> >> 5. The list forms consensus that the last round of your patch is good. Send
>> >>    it to the maintainer and cc the list.
>>
>> Yeah, but as far as I can tell, this is the _first_ round the list
>> sees this topic, which by definition would not have any consensus
>> ;-)
>
> I thought about it for over a year and still have no clue how we could
> teach GitGitGadget to Cc: you when it is appropriate, not without putting
> the burden on any human being.

That message is from November 2019, didn't you later fix this in January 2020 here: https://github.com/gitgitgadget/gitgitgadget/commit/b2221f4 ?

I.e. now users need to explicitly add "cc: gitster@pobox.com" to the description, no? So isn't in the same as for us who use the format-patch/send-email flow in this regard?

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 8 of 20 in “add--interactive: skip index refresh in reset patch mode”
  1. 0/1 add--interactive: skip index refresh in reset patch modeNika Layzell via GitGitGadget, Nov 23, 2019
  2. 1/1 add--interactive: skip index refresh in reset patch modeNika Layzell via GitGitGadget, Nov 23, 2019
  3. Junio C HamanoNov 24, 2019
  4. Johannes SchindelinNov 25, 2019
  5. Johannes SchindelinNov 25, 2019
  6. Junio C HamanoNov 26, 2019
  7. Cc'ing the Git maintainer on GitGitGadget contributions, was Re: [PATCH 0/1] add--interactive: skip index refresh in reset patch modeJohannes Schindelin, Jan 7, 2021
  8. Ævar Arnfjörð BjarmasonJan 7, 2021
  9. Johannes SchindelinJan 7, 2021
  10. Junio C HamanoJan 7, 2021
  11. Johannes SchindelinJan 8, 2021
  12. Junio C HamanoJan 8, 2021
  13. Johannes SchindelinJan 10, 2021
  14. Taylor BlauJan 8, 2021
  15. Johannes SchindelinJan 10, 2021
  16. Junio C HamanoJan 10, 2021
  17. Taylor BlauJan 11, 2021
  18. Junio C HamanoJan 12, 2021
  19. 胡哲宁Jan 14, 2021
  20. Junio C HamanoNov 26, 2019

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.