Date: Thu, 4 Mar 2004 22:51:20 -0800 From: Peter Wemm <peter@wemm.org> To: obrien@FreeBSD.org Cc: Peter Wemm <peter@FreeBSD.org> Subject: Re: PERFORCE change 48112 for review Message-ID: <200403042251.20025.peter@wemm.org> In-Reply-To: <20040305061315.GB58624@dragon.nuxi.com> References: <200403040613.i246DLgM015954@repoman.freebsd.org> <20040305061315.GB58624@dragon.nuxi.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday 04 March 2004 10:13 pm, David O'Brien wrote: > On Wed, Mar 03, 2004 at 10:13:21PM -0800, Peter Wemm wrote: > > http://perforce.freebsd.org/chv.cgi?CH=48112 > > > > Change 48112 by peter@peter_daintree on 2004/03/03 22:12:27 > > > > Umm. VESA doesn't exist on amd64. Sorry David. > > I know -- I still wanted as much of sys/i386/conf/NOTES enabled as > made [*]sense. > > -- David > [*] you might expand this as "semi-", "doesn't", or "". The problem is that we have some entries in options.amd64 that have no backing code. This stops config from detecting the error. Its important that we don't silently accept options that don't do anything. VESA and VESA_DEBUG merely cause an entry to be made in opt_vesa.h - but nothing ever #include's it. Allowing config to silently accept it means that people will get POLA when they wonder why the heck the VESA driver isn't working when the options are in their kernel and it didn't fail to compile. The fact that its not actually there is certainly a POLA violation. -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "All of this is for nothing if we don't go to the stars" - JMS/B5
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200403042251.20025.peter>