Date: Fri, 17 Mar 2006 12:20:37 -0800 From: Sam Leffler <sam@errno.com> To: John-Mark Gurney <gurney_j@resnet.uoregon.edu> Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, Dag-Erling Smorgrav <des@FreeBSD.org> Subject: Re: cvs commit: src/etc/rc.d Makefile kernel Message-ID: <441B1A15.60506@errno.com> In-Reply-To: <20060317191059.GK35129@funkthat.com> References: <200603171628.k2HGSCN8083438@repoman.freebsd.org> <20060317191059.GK35129@funkthat.com>
next in thread | previous in thread | raw e-mail | index | archive | help
John-Mark Gurney wrote:
> Dag-Erling Smorgrav wrote this message on Fri, Mar 17, 2006 at 16:28 +0000:
>> des 2006-03-17 16:28:12 UTC
>>
>> FreeBSD src repository
>>
>> Modified files:
>> etc/rc.d Makefile
>> Added files:
>> etc/rc.d kernel
>> Log:
>> This script symlinks /boot/kernel to the directory that contains the kernel
>> you booted from, unless /boot/kernel already exists and is not a symlink.
>> This should only affect people like me who juggle multiple kernels and
>> have KODIR = /boot/${KERN_IDENT} in /etc/make.conf to keep them apart.
>
> What part of the tree assumes that /boot/kernel exists? Shouldn't they
> be using a dirname'd version of kern.bootfile, which would make this
> change unnecessary?
>
Makefiles in modules and in built kernels do I believe. When I did the
recent make release+sysinstall mods there was a bunch of discussion
about whether /boot/kernel could be eliminated and/or made a symlink.
In the end I took the conservative route and mv'd the installed kernel
to /boot/kernel. For compatibility with des's change sysinstall could
just setup a symlink. However it might also be good to leave the
installed kernel around in case of a calamity (probably depends mostly
on disk space). Another interesting approach is to hack loader to do
something more intelligent when /boot/kernel is not present (e.g. Ed
Maste has mods to pick a up/smp kernel automatically).
Sam
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?441B1A15.60506>