Date: Wed, 23 Sep 1998 12:30:00 -0700 (PDT) From: dag-erli@ifi.uio.no (Dag-Erling C. =?iso-8859-1?Q?Sm=F8rgrav?= ) To: freebsd-bugs@FreeBSD.ORG Subject: Re: conf/8031: /etc/rc looks for /etc/sendmail.cf at startup Message-ID: <199809231930.MAA12485@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
The following reply was made to PR conf/8031; it has been noted by GNATS.
From: dag-erli@ifi.uio.no (Dag-Erling C. =?iso-8859-1?Q?Sm=F8rgrav?= )
To: Studded <Studded@dal.net>
Cc: Drew Derbyshire <ahd@kew.com>, freebsd-bugs@FreeBSD.ORG,
freebsd-gnats-submit@FreeBSD.ORG
Subject: Re: conf/8031: /etc/rc looks for /etc/sendmail.cf at startup
Date: 23 Sep 1998 21:20:15 +0200
Studded <Studded@dal.net> writes:
> This isn't directed towards Drew, but technically the RHS of the test
> should be "XYES" instead of how it is now. The '=' conditional compares
> strings, so the LHS is correct, and it should be the same on both sides.
> [...]
> > ! if [ "X${sendmail_enable}" = X"YES" ]; then
> > echo -n ' sendmail'; /usr/sbin/sendmail ${sendmail_flags}
> > fi
The X is totally unnecessary as long as you have quotes around the
variable to make sure you get an empty string (rather than a missing
argument) if it's undefined or empty.
[ "${sendmail_enable}" = YES ] &&
echo -n ' sendmail' && /usr/sbin/sendmail ${sendmail_flags}
Might as well throw in a ${sendmail_program} for good measure:
[ "${sendmail_enable}" = YES ] &&
echo -n ' sendmail' && ${sendmail_program} ${sendmail_flags}
I prefer to place the echo command last so that ' sendmail' is not
printed unless sendmail started successfully.
DES
--
Dag-Erling Smørgrav - dag-erli@ifi.uio.no
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-bugs" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199809231930.MAA12485>