rc.d/sendmail: Fix the rcorder block

[0mp]

Summary

rcorder(8) requires the rcorder block to be an uninterrupted sequence of
REQUIRE, PROVIDE, BEFORE, and KEYWORD lines. Having a comment in between
REQUIRE and KEYWORD makes rcorder stop parsing the block when it reaches
the comment. Fix that by moving the comment out from the rcorder block.

MFC after: 3 days

Test Plan

After the patch is applied, rcorder will report sendmail as having the shutdown keyword:

rcorder -k shutdown /etc/rc.d/* | grep sendmail

[markj]

This seems fine to me. Looks like it's been broken for a rather long time.

What exactly is the effect? Does sendmail actually need the shutdown keyword?

[bnovkov]

LGTM, approved.

[gshapiro]

In D46924#1072046, @markj wrote:

What exactly is the effect? Does sendmail actually need the shutdown keyword?

Yes, to ensure a clean shutdown of active SMTP connections (and writing any in memory queue files).

[christos]

Approved.

[0mp]

In D46924#1072191, @gshapiro wrote:

In D46924#1072046, @markj wrote:

What exactly is the effect? Does sendmail actually need the shutdown keyword?

Yes, to ensure a clean shutdown of active SMTP connections (and writing any in memory queue files).

Great info! Thanks!