From owner-cvs-all@FreeBSD.ORG Tue Jan 22 21:51:12 2008 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8A99B16A468 for ; Tue, 22 Jan 2008 21:51:12 +0000 (UTC) (envelope-from brix@FreeBSD.org) Received: from solow.pil.dk (relay.pil.dk [195.41.47.164]) by mx1.freebsd.org (Postfix) with ESMTP id 41F2613C447 for ; Tue, 22 Jan 2008 21:51:12 +0000 (UTC) (envelope-from brix@FreeBSD.org) Received: from tirith.brixandersen.dk (0x55534f5f.adsl.cybercity.dk [85.83.79.95]) by solow.pil.dk (Postfix) with ESMTP id 7993C1CC0D3; Tue, 22 Jan 2008 22:51:10 +0100 (CET) Received: by tirith.brixandersen.dk (Postfix, from userid 1001) id 863E311436; Tue, 22 Jan 2008 22:51:09 +0100 (CET) Date: Tue, 22 Jan 2008 22:51:09 +0100 From: Henrik Brix Andersen To: Rong-En Fan Message-ID: <20080122215109.GD21985@tirith.brixandersen.dk> Mail-Followup-To: Rong-En Fan , Doug Barton , ports-committers@FreeBSD.org, cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org References: <200801200837.m0K8bNX0081391@repoman.freebsd.org> <4795044E.6030006@FreeBSD.org> <20080122015108.GX4650@svm.csie.ntu.edu.tw> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="wxDdMuZNg1r63Hyj" Content-Disposition: inline In-Reply-To: <20080122015108.GX4650@svm.csie.ntu.edu.tw> X-PGP-Key: http://www.brixandersen.dk/files/HenrikBrixAndersen.asc User-Agent: Mutt/1.5.17 (2007-11-01) Cc: cvs-ports@FreeBSD.org, Doug Barton , cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/sysutils/LPRng Makefile distinfo pkg-descr pkg-plist ports/sysutils/LPRng/files lprng.in pkg-message.in X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Jan 2008 21:51:12 -0000 --wxDdMuZNg1r63Hyj Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 22, 2008 at 09:51:08AM +0800, Rong-En Fan wrote: > On Mon, Jan 21, 2008 at 12:45:02PM -0800, Doug Barton wrote: > > The script looks good, thanks for doing this! The only thing I would po= int=20 > > out is that the default _enable=3Dno is not needed, since rc.subr needs= to=20 > > see one of the values for "yes" in order to do anything. No harm leavin= g it=20 > > in, I just wanted to point out that it's not strictly necessary. >=20 > Thanks. I see most of our /etc/rc.d/* scripts does not have this. > But most ports' have as our Porter's Handbook have such example. I believe this is actually required for rc scripts supplied by ports. Scripts included in the base wont need it, since their default settings are already present in /etc/defaults/rc.conf. I just tried removing the default setting for '$bitlbee_enable' in the rc script for irc/bitlbee, I did recently. It then results in the following error when '$bitlbee_enable' is undefined: # /usr/local/etc/rc.d/bitlbee rcvar # bitlbee /usr/local/etc/rc.d/bitlbee: WARNING: $bitlbee_enable is not set properly -= see rc.conf(5). bitlbee_enable=3DNO Brix --=20 Henrik Brix Andersen --wxDdMuZNg1r63Hyj Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.7 (FreeBSD) Comment: GnuPG signed iD8DBQFHlmVMv+Q4flTiePgRAh7LAJ42Gp7sq66m387A0nxtpbi69Gnw0QCdH8m7 ZMhxwNceMgcSDQChMIMR8wo= =nef6 -----END PGP SIGNATURE----- --wxDdMuZNg1r63Hyj--