Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Jun 2008 11:56:13 +0300
From:      Mike Makonnen <mtm@wubethiopia.com>
To:        Doug Barton <dougb@FreeBSD.org>
Cc:        =?UTF-8?B?RGFnLUVybGluZyBTbcO4cmdyYXY=?= <des@des.no>, src-committers@freebsd.org, freebsd-rc@freebsd.org
Subject:   Re: svn commit: r179669 - head/sbin/rcorder
Message-ID:  <48562AAD.90900@wubethiopia.com>
In-Reply-To: <485179C8.9090802@FreeBSD.org>
References:  <200806090907.m5997wwM043284@svn.freebsd.org> <86hcc2vjnv.fsf@ds4.des.no> <484E2E05.5080804@wubethiopia.com> <485179C8.9090802@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Doug Barton wrote:
> Mike Makonnen wrote:
>> [ cc'ing -rc ]
>>
>> Dag-Erling Smørgrav wrote:
>>> Mike Makonnen <mtm@FreeBSD.org> writes:
>>>>   Document the misleading nature of the REQUIRE line. The patch in
>>>>   the PR has been heavily edited for style(9) and clarity. Mistakes 
>>>> are
>>>>   mine.
>>>
>>> How about s/REQUIRE/AFTER/, to mirror BEFORE?
>>
>> I would be in favor of that. Any objections?
>
> In some ways I think this would be a good change, although I confess I 
> have a yellow warning light in the "possibly gratuitous change" 
> column. My preference would be that we ask the netbsd guys about this 
> before we do it, either so that they can consider doing it also, or to 
> warn us about why it would be a bad idea.
It may seem gratuitous, but on the other hand if it prevents 
misunderstandings of the kind that spawned this thread in the first 
place, then I think it's worth doing. As far as NetBSD is concerned we 
can have compatibility by simply *not* removing support for "REQUIRED". 
We can simply list it as supported but deprecated in the documentation.

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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?48562AAD.90900>