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

Re: [PATCH] use -q instead of redirect to /dev/null for git update-index

From
BMBenjamin Meyer <ben@meyerhome.net>
Date
Mar 21, 2010, 17:02 UTC
Message-ID
<70E4FE83-0DC9-4AF1-BA20-BE6467FC8D07@meyerhome.net>
In-Reply-To
<7vy6hlvmoy.fsf@alter.siamese.dyndns.org>
Show 9 quoted lines
> The above can be said to other "/dev/null to -q" patches we saw on the
> list recently.  The saddest part of the story is that, the review cost
> (not necessarily spent by me, but time spent by other people reviewing
> patches is precious) is paid only to prevent a breakage like this patch
> tries to introduce from getting accepted, without much potential reward.
> Even if some of those patches turn out to be harmless, the only thing we
> would have bought with the cost of reviewing is that we made sure that the
> system would continue to work as before, but that is what we could have
> easily done by not even looking at the patches at all.
For what it is worth I got the task from the Git Janitor wiki page.  https://git.wiki.kernel.org/index.php/Janitor
Sorry I didn't include that in my changelog.
-Benjamin Meyer
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 4 in “use -q instead of redirect to /dev/null for git update-index”
  1. use -q instead of redirect to /dev/null for git update-indexRomain Bignon, Mar 21, 2010
  2. Junio C HamanoMar 21, 2010
  3. Benjamin MeyerMar 21, 2010
  4. Junio C HamanoMar 21, 2010

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.