Date: Tue, 03 Oct 2017 22:13:37 +0000 From: bugzilla-noreply@freebsd.org To: freebsd-rc@FreeBSD.org Subject: [Bug 216115] service -e showing sendmail enabled when it is not Message-ID: <bug-216115-4805-bc2gQ3VzSK@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-216115-4805@https.bugs.freebsd.org/bugzilla/> References: <bug-216115-4805@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D216115 Jilles Tjoelker <jilles@FreeBSD.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jilles@FreeBSD.org --- Comment #9 from Jilles Tjoelker <jilles@FreeBSD.org> --- The proposed patch basically executes status when executing any subcommand,= and then adds status and poll to the available subcommands when the daemon is running. This seems wrong. Although the if [ -n "$_pidcmd" ]; then part is definitely redundant, I thi= nk this condition should just be removed instead of changed. The idea is that _pidcmd is defined and status and poll are available iff a command or proce= ss name is defined (which may be done in various ways). What happens with /etc/rc.d/sendmail is that service(8)'s naive greps catch= the lines name=3D"sendmail_msp_queue" rcvar=3D"sendmail_msp_queue_enable" in addition to the intended lines name=3D"sendmail" rcvar=3D"sendmail_enable" and the msp_queue ones override the intended lines because they are last. Indenting the two msp_queue lines in /etc/rc.d/sendmail fixes the problem b= ut it is a rather ugly and obscure solution. --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-216115-4805-bc2gQ3VzSK>