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

Re: Reviews on mailing-list

From
Thiago Farina <tfransosi@gmail.com>
Date
Nov 11, 2012, 01:13 UTC
Message-ID
<CACnwZYekU0CYnqQT8L2siJbUsn=T9qowgth94TWc8KN472Ziag@mail.gmail.com>
In-Reply-To
<CAMP44s2AvaAs-dn_eoSFWoLHBQk5QrrV=zYTOpRi2nEywoua4w@mail.gmail.com>

On Sat, Nov 10, 2012 at 9:40 PM, Felipe Contreras <felipe.contreras@gmail.com> wrote:

Show 19 quoted lines
> On Sun, Nov 11, 2012 at 12:19 AM, Deniz Türkoglu <deniz@spotify.com> wrote:
>
>> This is my first mail to the git mailing list. I have been following
>> the list for some time now and I would like to suggest moving the
>> reviews out of the mailing list, for example to a gerrit instance, I
>> believe it would improve the commits and the mailing list. I have a
>> filter on 'PATCH', but I feel I miss some of the discussion, and
>> things that I would be interested in.
>>
>> I have spoken to Shawn Pearce (gerrit project lead, google) and he
>> said he is OK with hosting the gerrit instance.
>>
>> I would like to hear your thoughts on this.
>
> Personally I think reviews on the mailing list is far superior than
> any other review methods. I've even blogged about it and all the
> reasons[1]. Gerrit is better than bugzilla, but it still requires a
> web browser, and logging in.
>

Requiring a web browser is a huge requirement, ham?? How come that can be an impediment to move forward way of this awkward way of reviewing patches through email? Switching to Gerrit would mean everyone would be using the same tool instead of anyone using its own email client (gmail, mutt, thunderbird, whatever...) and having to figure out git format-patch, git send-email (--reply-to where?).

There are a lot of issues of having to use email for reviewing patches that I think Gerrit is a superior alternative.

And many people are arguing for it!
Let's move on...
Previous: Felipe ContrerasNext: Junio C Hamano
Message 3 of 17 in “Reviews on mailing-list”
  1. Deniz TürkogluNov 10, 2012
  2. Felipe ContrerasNov 10, 2012
  3. Thiago FarinaNov 11, 2012
  4. Junio C HamanoNov 11, 2012
  5. suvayu aliNov 11, 2012
  6. Nguyen Thai Ngoc DuyNov 11, 2012
  7. Felipe ContrerasNov 11, 2012
  8. Thiago FarinaNov 11, 2012
  9. Felipe ContrerasNov 11, 2012
  10. Krzysztof MazurNov 11, 2012
  11. David LangNov 11, 2012
  12. Deniz TürkogluNov 12, 2012
  13. David LangNov 12, 2012
  14. Nguyen Thai Ngoc DuyNov 13, 2012
  15. Deniz TürkogluNov 11, 2012
  16. Felipe ContrerasNov 11, 2012
  17. Ramkumar RamachandraNov 11, 2012

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.