svn commit: r293391 - stable/10

[Slawa Olhovchenkov]

On Thu, Jan 21, 2016 at 06:49:49PM -0800, NGie Cooper wrote:

> 
> > On Jan 21, 2016, at 18:25, Gleb Smirnoff <glebius at FreeBSD.org> wrote:
> > 
> > On Fri, Jan 22, 2016 at 05:18:28AM +0300, Slawa Olhovchenkov wrote:
> > S> > On Thu, Jan 21, 2016 at 4:29 PM, Slawa Olhovchenkov <slw at zxy.spb.ru> wrote:
> > S> > > On Thu, Jan 07, 2016 at 11:10:44PM +0000, Gleb Smirnoff wrote:
> > S> > >
> > S> > >> Author: glebius
> > S> > >> Date: Thu Jan  7 23:10:44 2016
> > S> > >> New Revision: 293391
> > S> > >> URL: https://svnweb.freebsd.org/changeset/base/293391
> > S> > >>
> > S> > >> Log:
> > S> > >>   Merge r293282 from head:
> > S> > >>    Provide knob NO_INSTALLEXTRAKERNELS. If defined, extra kernels in KERNCONF
> > S> > >>    won't be installed, only the first one would.
> > S> > >>
> > S> > >>   Turn NO_INSTALLEXTRAKERNELS=yes on stable/10 to preserve original
> > S> > >>   behaviour of not installing additional kernels built.
> > S> > >
> > S> > > What?!
> > S> > > By default additional kernels built will be installed.
> > S> > 
> > S> > Agreed, but they would overwrite one another, unless you used
> > S> > INSTKERNNAME appropriately.
> > S> 
> > S> I am use relese.sh with KERNEL="GENEROC VSTREAM" and nothing
> > S> overwritied.
> > S> INSTKERNNAME properly used with KERNCONF by Makefile.inc1
> > S> (kernel for first and kernel.${_kernel} for other.
> > S> 
> > S> Now relese.sh can't used with multiple kernels, this is POLA
> > S> violation.
> > 
> > You are confusing release.sh with regular build. Now in head regular
> > build behaves same as release.sh. But in stable/10 NO_INSTALLEXTRAKERNELS=1
> > to provide old behaviour.
> 
> It should be ?=, not = then? Now it's always on..

With ?= this is also break expected, used and documented behaivor for
relese.sh with multiple kernel.