Date: Fri, 01 Sep 2017 12:00:09 -0500 From: Josh Paetzel <jpaetzel@FreeBSD.org> To: svn-src-all@freebsd.org Subject: Re: svn commit: r323087 - in head/sys: amd64/conf arm64/conf i386/conf powerpc/conf riscv/conf sparc64/conf Message-ID: <1504285209.3899479.1092460000.413562B8@webmail.messagingengine.com> In-Reply-To: <1504284895.41612.29.camel@freebsd.org> References: <201709011554.v81FssRG083114@repo.freebsd.org> <CAG6CVpVnEejQdE-=SLyLU__PJpuFNmC03FHnnx1Vfnv=uZN20Q@mail.gmail.com> <1504284895.41612.29.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Ok, I'm going to revert it for now. There was a mistake in the commit, the commit message was wrong, and it sounds like we need more consensus. On Fri, Sep 1, 2017, at 11:54 AM, Ian Lepore wrote: > On Fri, 2017-09-01 at 09:06 -0700, Conrad Meyer wrote: > > Wait, why?=C2=A0=C2=A0The PR doesn't explain it any better than this co= mmit > > message. > >=20 > > On Fri, Sep 1, 2017 at 8:54 AM, Josh Paetzel <jpaetzel@freebsd.org> > > wrote: > > >=20 > > > Author: jpaetzel > > > Date: Fri Sep=C2=A0=C2=A01 15:54:53 2017 > > > New Revision: 323087 > > > URL: https://svnweb.freebsd.org/changeset/base/323087 > > >=20 > > > Log: > > > =C2=A0 Take options IPSEC out of GENERIC > > >=20 > > > =C2=A0 PR:=C2=A0=C2=A0=C2=A0220170 > > > =C2=A0 Submitted by: delphij > > > =C2=A0 Reviewed by:=C2=A0=C2=A0ae, glebius > > > =C2=A0 MFC after:=C2=A0=C2=A0=C2=A0=C2=A02 weeks > > > =C2=A0 Differential Revision:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0D11806 >=20 > The differential revision (which should be cited using the complete url > in the commit) seems to indicate that it's because it can be kldloaded. >=20 > This change misses the armv6 platform, where the option appears in > std.armv6, which gets included in GENERIC (and all other armv6 > configs). >=20 > This change seems to warrant an entry in UPDATING, because it's going > to catch a lot of people by surprise. =C2=A0Even people with custom kerne= ls, > if they follow the advice of the handbook and "include GENERIC" then > override things. >=20 > -- Ian > _______________________________________________ > svn-src-all@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-all > To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org" --=20 Thanks, Josh Paetzel
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1504285209.3899479.1092460000.413562B8>