svn commit: r331728 - in stable/11/etc: . rc.d

[Ian Lepore]

On Thu, 2018-03-29 at 06:20 -0700, Rodney W. Grimes wrote:
> > 
> > Author: araujo
> > Date: Thu Mar 29 04:51:07 2018
> > New Revision: 331728
> > URL: https://svnweb.freebsd.org/changeset/base/331728
> > 
> > Log:
> >   MFC r329817:
> I must of missed this when it landed in ^/head
> 
> > 
> >   The firewall_type is ignored if not set in rc.conf or rc.conf.local,
> >   after r190575 there is an option to call rc.firewall with the firewall_type
> >   passed in as an argument.
> >   
> >   Submitted by:	David P. Discher <dpd at dpdtech.com>
> >   Sponsored by:	iXsystems Inc.
> >   Differential Revision:	https://reviews.freebsd.org/D14286
> No one accepted it :-(.
> 

That's not a blocker for committing; plenty of time elapsed to allow
anyone to reject the change. IMO, even a flat-out rejection isn't a
blocker to committing except for things like random or crypto code that
require formal approval (but I'd certainly think hard about committing
if people rejected the change, and put some effort into finding a
compromise first).

There were comments added to the review, which makes citing the review
in the commit useful.  I usually also add a note such as '(timed out)'
after the url, but I've noticed that doing so ruins the automatic
closing of the review and requires you to manually abandon it instead.

-- Ian