Re: [PATCH] Make the exit code of add_file_to_index actually useful

Junio C Hamano, Sun, Mar 02, 2008 17:59:13 +0100:

Johannes Schindelin [off-list ref] writes:

quoted

On Sun, 2 Mar 2008, Alex Riesen wrote:

quoted

-			add_file_to_cache(path, verbose);
+			if (add_file_to_cache(path, verbose))
+				exit(1);

Does it really, really _have_ to be exit(1)?  I mean, now you block even 
the faintest chance that we can libify libgit.a by overriding die_routine.

I think Alex did so not to break the existing scripts that rely on these
dying, but it should have been exit(128) to really stay compatible.

I corrected the series to use die() again and rebased it off current
master (65ea3b8c). So it is more compatible with libification (does
not hinder it more than previos code) and keep the exit code.