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

Re: [PATCH] Make gc a builtin.

From
Brian Gernhardt <benji@silverinsanity.com>
Date
Mar 14, 2007, 01:37 UTC
Message-ID
<7F86A2D8-0407-4663-A97F-7D003F370E0F@silverinsanity.com>
In-Reply-To
<Pine.LNX.4.63.0703140203250.22628@wbgn013.biozentrum.uni-wuerzburg.de>
On Mar 13, 2007, at 9:05 PM, Johannes Schindelin wrote:
> If you say "return error(...);", there is _no_ way that multiple error
> messages are printed out.

Except that cmd_gc() is littered with run_command* calls, which fork off a subprocess to do the heavy lifting. So if git-repack fails an error will be printed by that process, making the error() call redundant. (If I'm understanding things correctly.)

~~ Brian
Previous: Johannes Schindelin
Message 3 of 3 in “Make gc a builtin.”
  1. Make gc a builtin.James Bowes, Mar 13, 2007
  2. Johannes SchindelinMar 14, 2007
  3. Brian GernhardtMar 14, 2007

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.