From owner-cvs-src@FreeBSD.ORG  Thu Nov 10 10:40:16 2005
Return-Path: <owner-cvs-src@FreeBSD.ORG>
X-Original-To: cvs-src@FreeBSD.org
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 4116C16A41F;
	Thu, 10 Nov 2005 10:40:16 +0000 (GMT) (envelope-from rse@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 0698143D46;
	Thu, 10 Nov 2005 10:40:16 +0000 (GMT) (envelope-from rse@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id jAAAeFOD068389;
	Thu, 10 Nov 2005 10:40:15 GMT (envelope-from rse@repoman.freebsd.org)
Received: (from rse@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id jAAAeFdu068388;
	Thu, 10 Nov 2005 10:40:15 GMT (envelope-from rse)
Message-Id: <200511101040.jAAAeFdu068388@repoman.freebsd.org>
From: "Ralf S. Engelschall" <rse@FreeBSD.org>
Date: Thu, 10 Nov 2005 10:40:15 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/etc/rc.d pf
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 10 Nov 2005 10:40:16 -0000

rse         2005-11-10 10:40:15 UTC

  FreeBSD src repository

  Modified files:
    etc/rc.d             pf 
  Log:
  Backout r1.11...
  
  > >   There is no need to explicitly add "status" to $extra_commands in
  > >   the /etc/rc.d/pf script as it is implicitly added by /etc/rc.subr's
  > >   run_rc_command() because of the existing $pf_program.
  > >
  > >   Submitted by:   Christoph Schug <chris@schug.net>
  
  ...because as yar@ points out: "[...] you were relying on evil
  side-effects of the variable being named *_program. hose side-effect
  have been eliminated since rc.subr rev. 1.42. [...] The point is that
  the default "status" method is for rc.d scripts that handle startup and
  shutdown of conventional daemons, and not for custom tasks like the pf
  case."
  
  The change is still valid in RELENG_6 (and still doesn't have to be
  backed out) as long as rc.subr:r1.42 is not MFC'ed to RELENG_6, too.
  
  Revision  Changes    Path
  1.12      +1 -1      src/etc/rc.d/pf