Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 26 May 2016 11:28:33 -0700
From:      John Baldwin <jhb@freebsd.org>
To:        Hans Petter Selasky <hps@selasky.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r299393 - in head: . share/mk
Message-ID:  <6335113.N7gIF2Maer@ralph.baldwin.cx>
In-Reply-To: <08cfa0ac-ad92-c8f9-2141-e72088b17d41@selasky.org>
References:  <201605102232.u4AMWNvp000287@repo.freebsd.org> <08cfa0ac-ad92-c8f9-2141-e72088b17d41@selasky.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday, May 25, 2016 03:27:07 PM Hans Petter Selasky wrote:
> On 05/11/16 00:32, John Baldwin wrote:
> > Author: jhb
> > Date: Tue May 10 22:32:23 2016
> > New Revision: 299393
> > URL: https://svnweb.freebsd.org/changeset/base/299393
> >
> > Log:
> >   Change the default installation directory for modules to /boot/modules.
> >
> >   Kernel installs always override KMODDIR when installing modules, so
> >   this default setting is only used for standalone module builds.  Many
> >   out-of-tree modules manually override KMODDIR already to avoid placing
> >   modules in /boot/kernel.  This now makes that behavior the default.
> >
> >   Discussed on:	arch@
> >   Reviewed by:	imp
> >   Relnotes:	yes
> >
> 
> Just a small nit:
> 
> There is a comment in bsd.own.mk which needs updating too:
> 
> > #
> > # KMODDIR       Base path for loadable kernel modules
> > #               (see kld(4)). [/boot/kernel]

Oops, thanks.  Fixed.

-- 
John Baldwin



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