sendmail queue runner starting multiple times...

[Hiroki Sato]

John-Mark Gurney <jmg at funkthat.com> wrote
  in <20130916214901.GE68682 at funkthat.com>:

jm> I noticed that if you start sendmail, it doesn't detect that the
jm> mail queue runner is already running and starts it up multiple times...
jm>
jm> My proposed patch is:
jm> --- /usr/src/etc/rc.d/sendmail  2013-01-12 21:34:02.000000000 -0800
jm> +++ /tmp/sendmail.rc    2013-09-16 14:45:21.000000000 -0700
jm> @@ -93,7 +93,7 @@
jm>
jm>  name="sendmail_clientmqueue"
jm>  rcvar="sendmail_msp_queue_enable"
jm> -start_cmd="${command} ${sendmail_msp_queue_flags}"
jm>  pidfile="${sendmail_mspq_pidfile:-/var/spool/clientmqueue/sm-client.pid}"
jm> +start_cmd="[ -z \"`check_pidfile "${pidfile}" "${command}"`\" ] && ${command} ${sendmail_msp_queue_flags} && _run_rc_postcmd"
jm>  required_files="/etc/mail/submit.cf"
jm>  run_rc_command "$1"
jm>
jm>
jm> This is because setting start_cmd disables the pid checks, so we need
jm> to add them back...  This patch won't print out the already running
jm> bit, but we also don't print out the fact that we are starting it, it
jm> doesn't seem like that big of a loss..
jm>
jm> Is there a better way?  If someone else doesn't produce a better patch,
jm> I'll get re's permission to commit in few days...
jm>
jm> Thanks.
jm>
jm> P.S. Please CC me, I'm not on the list.

 Does the attached patch fix this issue?

-- Hiroki
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rc_d_sendmail.20130916-1.diff
Type: text/x-patch
Size: 932 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-rc/attachments/20130917/98ca2f39/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-rc/attachments/20130917/98ca2f39/attachment.sig>