From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 06 Jan 2009 08:19:09 GMT
Subject: Re: [PATCH v2 resend] git.c: make autocorrected aliases work
Message-ID: <7v7i58kfsy.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20090104172833.GA7139@myhost>

Alexander Potashev <aspotashev@gmail.com> writes:

>> @@ -479,31 +501,22 @@ int main(int argc, const char **argv)
>> ...
>> +		was_alias = run_argv(&argc, &argv);
>> +		if (errno != ENOENT)
>>  			break;
>> +		if (was_alias) {
>>  			fprintf(stderr, "Expansion of alias '%s' failed; "
>>  				"'%s' is not a git-command\n",
>>  				cmd, argv[0]);
>>  			exit(1);
>
> Why not using 'die' here?

The code is in the context, and I do not think it is a good idea to
conflate such a change to a patch that wants to add aliases auto
correction.

While I do not think it matters too much in practice (unless existing
scripts that runs git depends on the exact error status value), there are
two differences: the message will say "fatal: " in front, and the command
exits with 128 not with 1.