From: Brian Gernhardt <benji@silverinsanity.com>
Date: Wed, 14 Mar 2007 01:37:54 GMT
Subject: Re: [PATCH] Make gc a builtin.
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