From owner-freebsd-rc@FreeBSD.ORG Sat Dec 10 19:42:35 2005 Return-Path: X-Original-To: freebsd-rc@FreeBSD.org Delivered-To: freebsd-rc@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C2E7716A41F for ; Sat, 10 Dec 2005 19:42:35 +0000 (GMT) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mail2.fluidhosting.com [204.14.90.12]) by mx1.FreeBSD.org (Postfix) with SMTP id 33D1143D58 for ; Sat, 10 Dec 2005 19:42:35 +0000 (GMT) (envelope-from dougb@FreeBSD.org) Received: (qmail 26266 invoked by uid 399); 10 Dec 2005 19:42:34 -0000 Received: from localhost (HELO ?192.168.1.101?) (dougb@dougbarton.us@127.0.0.1) by localhost with SMTP; 10 Dec 2005 19:42:34 -0000 Message-ID: <439B2FA8.2050704@FreeBSD.org> Date: Sat, 10 Dec 2005 11:42:32 -0800 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 1.5 (X11/20051203) MIME-Version: 1.0 To: Brooks Davis References: <4398AB6A.3000209@FreeBSD.org> <20051208222013.GB5759@odin.ac.hmc.edu> <43996043.8020609@FreeBSD.org> <20051209193830.GA15097@odin.ac.hmc.edu> <439A9857.9090003@FreeBSD.org> <20051210182100.GA12946@odin.ac.hmc.edu> In-Reply-To: <20051210182100.GA12946@odin.ac.hmc.edu> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: freebsd-rc@FreeBSD.org Subject: Re: For Review: Latest update to rcorder patch 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, 10 Dec 2005 19:42:35 -0000 Brooks Davis wrote: > On Sat, Dec 10, 2005 at 12:56:55AM -0800, Doug Barton wrote: Thanks for the review on the man page stuff. >>> OK. I might make rcconf.sh into a no-op that provides rcconf in the >>> first commit and remove it for real in a day or so to make it easier to >>> debug any inadvertent issues that result from increasing the number of >>> scripts with no dependencies. >> Yeah, not only is this a factor, but I also failed to take into account >> another critical element. rcorder needs at least one script without any >> REQUIRE lines to know where to start. Right now that's rcconf.sh by >> design, but I think we need to replace that, rather than just delete >> rcconf.sh. This is one of the few times when I think a dummy dependency >> would be useful. I propose that I add a dummy dependency called START, and >> change everything that currently depends on rcconf to depend on START >> instead. Sound good? > > I'm not sure you actually need a dummy. NetBSD just lets early things > have no REQUIRE lines (for instance cdd only contains "PROVIDE: > disks"). There's no requirement that the tree have a single top. Hrrrm ... ok. I removed rcconf.sh and all it's REQUIRE lines, and nothing changed in the order. David O'Brien added a BEFORE: initrandom to dumpon so that it would be easier to debug panics related to it, and I don't see any reason to change that, so I think we're set there. >> BTW, I noticed that there are several scripts that have no REQUIRE or >> nostart. Should those be updated? They are: ccd, gbde, mixer, and >> securelevel. > > mixer is broken and should REQUIRE on cleanvar. The ccd script probably > doesn't depend on anything but the obsolete rcconf.sh (which it > currently gets from initrandom's BEFORE: disks), but gbde may depend on > initrandom (I haven't read the code). I might just let initrandom take > the top slot. securelevel is just plain wrong in the current setup. If > it's going to do any good, it needs to come much earlier (NetBSD has it > BEFORE: DAEMON which makes sense if it's supposed to secure your system > from bad guys.) Right now it's in the noise at the end of the list due > to a whole lot of IMO rather bogus BEFORE: securelevel entries. Ok. Changing mixer is an easy call (and ironically pushes it further down the order), but I'm not comfortable changing the others. Things work as they are, even if it is by accident, and I don't want to be the one who breaks it. :) Doug -- This .signature sanitized for your protection