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

Re: Reviews on mailing-list

From
DLDavid Lang <david@lang.hm>
Date
Nov 11, 2012, 21:15 UTC
Message-ID
<alpine.DEB.2.02.1211111313140.19687@nftneq.ynat.uz>
In-Reply-To
<20121111171451.GA26695@shrek.podlesie.net>
On Sun, 11 Nov 2012, Krzysztof Mazur wrote:
Show 9 quoted lines
> On Sun, Nov 11, 2012 at 11:09:36AM -0200, Thiago Farina wrote:
>> On Sun, Nov 11, 2012 at 10:14 AM, Felipe Contreras
>> <felipe.contreras@gmail.com> wrote:
>>> Requiring everyone to use a web browser would limit the amount of ways
>>> people can review patches.
>> I don't see that as a limitation as I think everyone has access to a
>> web browser these days, don't have?
>
> Almost everyone, but not everyone likes using web browser.
Using a web browser requires connectivity at the time you are doing the review.

Mailing list based reviews can be done at times when you don't have connectivity.

Coincidently, one of the common times when people don't have connectivity is when they are traveling, and this can be a good time to do things like code review.

David Lang
Previous: Krzysztof MazurNext: Deniz Türkoglu
Message 11 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.