Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: $Revision: 1.6.2.13 $; site uiucdcs.UUCP
Path: utzoo!linus!vaxine!wjh12!genrad!grkermit!masscomp!clyde!burl!ulysses!mhuxl!ihnp4!inuxc!pur-ee!uiucdcs!liberte
From: liberte@uiucdcs.UUCP
Newsgroups: net.bugs.4bsd
Subject: Re: Re: sendbug(1) makes terrible assump - (nf)
Message-ID: <8200018@uiucdcs.UUCP>
Date: Mon, 23-Apr-84 17:53:00 EST
Article-I.D.: uiucdcs.8200018
Posted: Mon Apr 23 17:53:00 1984
Date-Received: Thu, 26-Apr-84 01:17:34 EST
References: <1509@vax4.UUCP>
Lines: 26
Nf-ID: #R:vax4:-150900:uiucdcs:8200018:000:765
Nf-From: uiucdcs!liberte    Apr 23 16:53:00 1984

#R:vax4:-150900:uiucdcs:8200018:000:765
uiucdcs!liberte    Apr 23 16:53:00 1984

The previous suggestion to check $EDITOR and $VISUAL rather than
always using vi has a problem.  In checking them, they must already
be defined.  Use the following instead:

	replace the line...
		/usr/ucb/vi /tmp/bug$$
	with...
		if ($?VISUAL) then
			set editor = $VISUAL
		else if ($?EDITOR) then
			set editor = $EDITOR
		else set editor = /usr/ucb/vi
		endif
		$editor /tmp/bug$$

I also noticed that an expression such as:
	if ($?VISUAL && $VISUAL == "vi") echo It works.
doesnt work.  Although the manual says: "the operators are similar to those
of C," full expression evaluation is used and an error is given if $VISUAL
is not defined.

Daniel LaLiberte,  U of Illinois, Urbana-Champaign, Computer Science
{moderation in all things - including moderation}