Date: Wed, 23 Sep 1998 10:58:45 -0700 From: Studded <Studded@dal.net> To: Drew Derbyshire <ahd@kew.com> Cc: freebsd-bugs@FreeBSD.ORG, freebsd-gnats-submit@FreeBSD.ORG Subject: Re: conf/8031: /etc/rc looks for /etc/sendmail.cf at startup Message-ID: <360936D5.9DAE1280@dal.net> References: <199809231210.FAA18954@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Drew Derbyshire wrote:
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.
There are numerous examples of this error. Although personally I would
prefer something that is a little more user friendly, like:
case "${sendmail_enable}" in
YES | yes )
echo -n ' sendmail'
/usr/sbin/sendmail "${sendmail_flags}"
;;
esac
I'd be happy to submit a patch if it had a chance of being committed. :)
> ! if [ "X${sendmail_enable}" = X"YES" ]; then
> echo -n ' sendmail'; /usr/sbin/sendmail ${sendmail_flags}
> fi
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?360936D5.9DAE1280>