Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Jun 2008 15:04:41 +0200
From:      Kris Kennaway <kris@FreeBSD.org>
To:        Pawel Jakub Dawidek <pjd@FreeBSD.org>
Cc:        Brooks Davis <brooks@FreeBSD.org>, freebsd-rc@FreeBSD.org, src-committers@FreeBSD.org, Dag-Erling Sm??rgrav <des@des.no>
Subject:   Re: svn commit: r179669 - head/sbin/rcorder
Message-ID:  <484FCD69.2080504@FreeBSD.org>
In-Reply-To: <20080611121338.GA2098@garage.freebsd.pl>
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>

next in thread | previous in thread | raw e-mail | index | archive | help
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 <mtm@wubethiopia.com> writes:
>>>>> Dag-Erling Sm??rgrav <des@des.no> 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).
> 
> 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.

Yes, rc.d changes are a big PITA to mergemaster.  I always ^C, then do
cd /usr/src/etc/rc.d; make install, then start over.  It would be nice 
if that could be automated.

Kris




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?484FCD69.2080504>