Re: CVS commit: src/etc

[Jean-Yves Migeon <jym%NetBSD.org@localhost>]

On 06.09.2011 12:54, Thomas Klausner wrote:
> On Mon, Aug 22, 2011 at 06:54:07PM +0000, Jean-Yves Migeon wrote:
>> Module Name: src
>> Committed By:        jym
>> Date:                Mon Aug 22 18:54:06 UTC 2011
>>
>> [snip]
>>
>> Log Message:
>> Modify etc/defaults/Makefile so that architectures can specify an additional
>> rc.conf file. This one should reside under etc/etc.${MACHINE}/, and will
>> get automatically appended to etc/defaults/rc.conf at build time if present.
>>
>>[snip]
>>
>> See also 
>> http://mail-index.netbsd.org/tech-userlevel/2011/07/25/msg005246.html
> 
> In a snapshot from a few hours ago I installed, "postinstall fix"
> without any other arguments removed the architecture specific section
> (the rc.conf included in the build output directory is fine).
>  Thomas

Hmmm, /etc/defaults/rc.conf should be transparent to postinstall(8), as
it gets installed via <bsd.files.mk> during build.sh.

If the source ('-s' of postinstall(8)) is not specified at command line,
according to code it takes "/usr/src" as default. Do you have a
stale/old rc.conf(5) file in there, /usr/src/etc/defaults/rc.conf?

Now, if this case should be handled properly is another question. There
are multiple solutions, all of them end up tweaking postinstall(8):
- raise a warning when overriding certain configuration files with older
versions (perhaps by using the CVS tags?)
- make source an explicit arg to etcupdate/postinstall
- something else?

-- 
Jean-Yves Migeon
jym%NetBSD.org@localhost