[Bug 216115] service -e showing sendmail enabled when it is not

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216115

Jilles Tjoelker <jilles at FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jilles at FreeBSD.org

--- Comment #9 from Jilles Tjoelker <jilles at 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.