Re: default editor

[Marco d'Itri <md@Linux.IT>]

On Mar 19, Will Yardley <you@aredumb.com> wrote:

[FYI: since somebody asked about it, bidirectional gating will be
enabled for all linux.debian.* newsgroups hopefully before the end of
the month, as soon as I can change what is needed in the software.
News->mail gating for linux.debian.user will be enabled right now.]

 >Yes, but mutt (normally) obeys VISUAL if present - it's only the Debian
 >package which seems not to.
Maybe you think this because the debian package is a 1.5 snaphost, and
handling of $EDITOR and $VISUAL has changed since 1.4 (something else
needed to be fixed, maybe this broken something too...).

I will *not* further discuss this with the mutt upstream unless
somebody will provide reference to authoritative documentation about the
correct semantics of $EDITOR and $VISUAL (I could not find any) and a
patch implementing this.

 >+  Editor = safe_strdup ((p = getenv ("EDITOR")) ? p : "/usr/bin/editor");
 >   Visual = safe_strdup ((p = getenv ("VISUAL")) ? p : Editor);
 >
 >I'd think that VISUAL (if present) should override /usr/bin/editor (if
 >present), however this seems not to be the case.
Read again the code: it does.
If you think that some parts of mutt use Editor while it should use
Visual instead please provide a patch with plenty of justifications
(I do not like changing the default hardcoded behaviour of a package
like mutt).

-- 
ciao,
Marco