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

Jilles Tjoelker <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #9 from Jilles Tjoelker <[email protected]> ---
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.
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-rc
To unsubscribe, send any mail to "[email protected]"

Reply via email to