From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 14 Apr 2009 08:51:35 GMT
Subject: Re: [PATCH RFC3 08/13] send-email: Remove superfluous `my $editor = ...'
Message-ID: <7v3acb7h48.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20090414061645.GB3948@atjola.homenet>

Björn Steinbrink <B.Steinbrink@gmx.de> writes:

> On 2009.04.13 16:39:52 -0700, Stephen Boyd wrote:
>> Michael Witten wrote:
>>> -	my $editor = $ENV{GIT_EDITOR} || Git::config(@repo, "core.editor") || $ENV{VISUAL} || $ENV{EDITOR} || "vi";
>>
>> I don't want to start an editor war, but why is 'vi' here? It seems
>> that one of the previous four should be set at all times, correct?
>
> Probably because that's how the rest of git works, too. See core.editor
> in git-config(1) or launch_editor() in editor.c if you don't trust the
> docs.
>
> Oh, and I seem to rely on that behaviour :-)

The git specific parts are obviously our invention, but $VISUAL then
$EDITOR and finally as the last resort falling back on vi has been a
longstanding UNIXy tradition that is shared by many tools.