svn commit: r293115 - head/etc

[Warner Losh]

On Mon, Jan 4, 2016 at 10:00 AM, Colin Percival <cperciva at tarsnap.com>
wrote:

> On 01/03/16 11:18, Warner Losh wrote:
> > Log:
> >   Use /bin/rm to remove /firstboot*. Otherwise rm -i alias is picked
> >   up and can cause issues on boot with the prompts.
>
> Huh, I never realized that could be a problem.


It can, but it was a false positive here. /bin/sh always sources them,
so if you'd added an alias, it would be hit here.


>
> >   Fix the read-only
> >   root case with horrible kludge of mounting rw removing the files, then
> >   mounting ro.
>
> The solution I intended when I introduced this (and used elsewhere) was to
> set $firstboot_sentinel in /etc(/defaults)?/rc.conf.  This case is
> precisely
> why it's a shell variable, in fact.


Except that's not exactly useful. NanoBSD boots with no filesystems writable
that are permanent.  So I could set it to /var/firstboot or something like
that,
and the error would go away. However, that wouldn't solve the problem
because /var is repopulated from base seed files every boot with NanoBSD
so we'd get firstboot behavior on every single boot. Or, we could remount
/ rw and remove the file and remount it ro when a read-only root was
requested.

I wondered to myself why we didn't use the same mechanism as nextboot
for this feature. Do you know?

Warner