From owner-freebsd-rc@FreeBSD.ORG Wed Jun 11 14:46:52 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 0A16B1065670; Wed, 11 Jun 2008 14:46:52 +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 CC8648FC16; Wed, 11 Jun 2008 14:46:51 +0000 (UTC) (envelope-from mtm@wubethiopia.com) Received: from rogue.mike.lan (unknown [213.55.87.92]) by dire.wubethiopia.com (Postfix) with ESMTPSA id E07E74FD966E; Wed, 11 Jun 2008 14:46:39 +0000 (UTC) Message-ID: <484FE658.6070008@wubethiopia.com> Date: Wed, 11 Jun 2008 17:51:04 +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> <484FCFE3.7030107@wubethiopia.com> <20080611141048.GC2098@garage.freebsd.pl> In-Reply-To: <20080611141048.GC2098@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 14:46:52 -0000 Pawel Jakub Dawidek wrote: > On Wed, Jun 11, 2008 at 04:15:15PM +0300, Mike Makonnen wrote: > [...] >> 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). > [...] >> 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. > > I thought it doesn't work for the first mergemaster(8) run. Am I wrong? > No, you are correct. I think I misunderstood you the first time. Are you suggesting to generate the md5 sum during buildworl/installworld time, and then have the bsd.*.mk glue auto-upgrade files based on this? Yes, I see how this would avoid mergemaster's problem of needing to prime its "cache". However, this stops working if the user modifies the file. I believe, in mergemaster's case if the user modifies the file you are asked to manually upgrade it the next time you run mergemaster, but after that it goes back to automatically updating it. I suppose mergemaster(8) could be modified to take advantage of this to prime it's cache before the first run, and then we could have the best of both worlds. This bears thinking on... Never the less, I think enabling the -i and -U options by default in mergemaster(8) (see other email sent to dougb, -rc, and -current) would go a long way to addressing your concerns regarding the extra churn involved in updating rc.d files. 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