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=216115 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 think 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 process 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="sendmail_msp_queue" rcvar="sendmail_msp_queue_enable" in addition to the intended lines name="sendmail" rcvar="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 but it is a rather ugly and obscure solution. -- 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>