From owner-freebsd-rc@FreeBSD.ORG Wed Jun 11 13:10:37 2008 Return-Path: Delivered-To: freebsd-rc@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8D53B1065671; Wed, 11 Jun 2008 13:10:37 +0000 (UTC) (envelope-from mtm@wubethiopia.com) Received: from dire.wubethiopia.com (j071.v.rootbsd.net [208.79.82.223]) by mx1.freebsd.org (Postfix) with ESMTP id 29B128FC0C; Wed, 11 Jun 2008 13:10:37 +0000 (UTC) (envelope-from mtm@wubethiopia.com) Received: from rogue.mike.lan (unknown [213.55.82.68]) by dire.wubethiopia.com (Postfix) with ESMTPSA id 9CB954FD966E; Wed, 11 Jun 2008 13:10:21 +0000 (UTC) Message-ID: <484FCFE3.7030107@wubethiopia.com> Date: Wed, 11 Jun 2008 16:15:15 +0300 From: Mike Makonnen User-Agent: Thunderbird 2.0.0.12 (X11/20080323) MIME-Version: 1.0 To: Pawel Jakub Dawidek References: <200806090907.m5997wwM043284@svn.freebsd.org> <86hcc2vjnv.fsf@ds4.des.no> <484E2E05.5080804@wubethiopia.com> <86lk1du3au.fsf@ds4.des.no> <20080610145350.GA19374@lor.one-eyed-alien.net> <484FB5DE.8060001@wubethiopia.com> <20080611121338.GA2098@garage.freebsd.pl> In-Reply-To: <20080611121338.GA2098@garage.freebsd.pl> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Dag-Erling Sm??rgrav , Brooks Davis , freebsd-rc@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r179669 - head/sbin/rcorder 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: Wed, 11 Jun 2008 13:10:37 -0000 Pawel Jakub Dawidek wrote: > On Wed, Jun 11, 2008 at 02:24:14PM +0300, Mike Makonnen wrote: >> Brooks Davis wrote: >>> On Tue, Jun 10, 2008 at 10:01:13AM +0200, Dag-Erling Sm??rgrav wrote: >>>> Mike Makonnen writes: >>>>> Dag-Erling Sm??rgrav writes: >>>>>> How about s/REQUIRE/AFTER/, to mirror BEFORE? >>>>> I would be in favor of that. Any objections? >>>> (obviously, we'd have to keep REQUIRE for backward compat, and perhaps >>>> emit a warning when we come across a script that uses it) >>> I think we'd want to support REQUIRE quietly until 6.x and 7.x releases >>> that >>> support AFTER have been shipped for some time and a sweep of ports has been >>> done. After that we could consider a warning in HEAD. >>> >>> That said, I'm not yet totally convinced (still thinking about it) given >>> the level of churn involved. >> Actually, I was thinking that we leave REQUIRE in there indefinitely for >> backwards compatibility and avoiding gratuitous differences with >> NetBSD and DragonFly. It doesn't cost us anything but is a net benefit >> to our users. We can simply discourage its use in the documentation. As >> far as churn is concerned: I don't think it's that big a deal >> (especially if you use mergemaster -U). The patch is actually pretty >> simple (see attachment). > > Just a note from a user perspective. Every time I install FreeBSD and I > upgrade it to the latest release, I've to spend a lot of time to > manually install all the changes like the above (I'm not used to -U > option, but I also think it doesn't work for the first time.). > And believe me, this is sooo huge PITA, especially if you have to > install more than one system (but not many enough to automate it). Which is exactly why mergemaster(8) has a -U option (it's such a time-saver I don't understand why it's not enabled by default). > If there really have to be changes like that, please consider installing > rc.d/ script as a part of the installworld target. More or less what > kib@ suggested the other day. The thing I don't like about it is that it would blow away any local modifications to those files. A lot of people are used to editing these files for major/minor local tweaks (in fact, I do it myself some times) and I think simply blowing these files away would inconvenience a lot of users. > One of the things I was thinking about how to detect if the script was > manually edited by the user is to keep md5 of the script in it. Of > course we have to exclude one line. eg. > Again: why re-invent the wheel? The mergemaster(8) command already does this, except that the md5 sum doesn't live in the file, but in /var/db/mergemaster.mtree. Cheers. -- Mike Makonnen | GPG-KEY: http://people.freebsd.org/~mtm/mtm.asc mtm @ FreeBSD.Org | AC7B 5672 2D11 F4D0 EBF8 5279 5359 2B82 7CD4 1F55 FreeBSD | http://www.freebsd.org