From owner-freebsd-rc@FreeBSD.ORG Sat Jan 7 01:06:00 2012 Return-Path: Delivered-To: freebsd-rc@FreeBSD.org Received: from mx2.freebsd.org (mx2.freebsd.org [IPv6:2001:4f8:fff6::35]) by hub.freebsd.org (Postfix) with ESMTP id 72D981065670; Sat, 7 Jan 2012 01:06:00 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from 172-17-198-245.globalsuite.net (hub.freebsd.org [IPv6:2001:4f8:fff6::36]) by mx2.freebsd.org (Postfix) with ESMTP id 76DE114F80F; Sat, 7 Jan 2012 01:05:59 +0000 (UTC) Message-ID: <4F079A76.3030306@FreeBSD.org> Date: Fri, 06 Jan 2012 17:05:58 -0800 From: Doug Barton Organization: http://SupersetSolutions.com/ User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20111222 Thunderbird/9.0 MIME-Version: 1.0 To: Pawel Jakub Dawidek References: <20120106141301.GE1693@garage.freebsd.pl> In-Reply-To: <20120106141301.GE1693@garage.freebsd.pl> X-Enigmail-Version: undefined OpenPGP: id=1A1ABC84 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: freebsd-rc@FreeBSD.org Subject: Re: Making use of set_rcvar. X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 07 Jan 2012 01:06:00 -0000 On 01/06/2012 06:13, Pawel Jakub Dawidek wrote: > Any objections? > > http://people.freebsd.org/~pjd/patches/set_rcvar.patch > > This patch only changes scripts where set_rcvar can be used with no > arguments. Please don't do this. Jilles already pointed out the important reason, it adds pointless forks. I suggested a long time ago that we remove set_rcvar altogether but I got a lot of resistance to it, and never pursued it. Perhaps it's time to revisit that. In regards to your concern about not having to type the name of the script when copying to create a new one, you can't really be serious about that? If so, let me introduce you to this revolutionary thing called "search and replace." If you really need to make this simpler for copy/paste purposes in http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/rc-scripts.html I recommend using: rcvar=${name}_enable The microsecond it takes to dereference that is an acceptable cost to making the copy/paste issue more palatable. This is particularly important in the ports area since there is so much copy/paste that goes on in that sphere. Doug -- You can observe a lot just by watching. -- Yogi Berra Breadth of IT experience, and depth of knowledge in the DNS. Yours for the right price. :) http://SupersetSolutions.com/