Date: Sun, 05 Jul 2015 14:05:30 -0600 From: Ian Lepore <ian@freebsd.org> To: John-Mark Gurney <jmg@funkthat.com> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r285160 - head/sys/conf Message-ID: <1436126730.1191.4.camel@freebsd.org> In-Reply-To: <20150705194954.GA8523@funkthat.com> References: <201507051415.t65EFxWd044357@repo.freebsd.org> <20150705194954.GA8523@funkthat.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 2015-07-05 at 12:49 -0700, John-Mark Gurney wrote: > Ian Lepore wrote this message on Sun, Jul 05, 2015 at 14:15 +0000: > > Author: ian > > Date: Sun Jul 5 14:15:58 2015 > > New Revision: 285160 > > URL: https://svnweb.freebsd.org/changeset/base/285160 > > > > Log: > > Ensure all the required files get built when you include the IPSEC option. > > Umm.. This looks more like that IPSEC should require that the crypto > device be specified instead of fixing it this way... I considered that, but rejected it given the presence of crypto | ipsec on some of the files already, but not a sufficiency of them to actually build sucessfully. Heavily my thinking that was a personal belief that if you say "option foo" it should just work without you needing to know other (likely not well documented) prerequisite options also need to be set. -- Ian
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1436126730.1191.4.camel>