Date: Fri, 4 Nov 2005 16:33:46 -0800 From: Peter Wemm <peter@wemm.org> To: Ceri Davies <ceri@submonkey.net> Cc: Warner Losh <imp@freebsd.org>, src-committers@freebsd.org, Ruslan Ermilov <ru@freebsd.org>, cvs-all@freebsd.org, cvs-src@freebsd.org Subject: Re: cvs commit: src/usr.sbin/config configvers.h Message-ID: <200511041633.47778.peter@wemm.org> In-Reply-To: <20051104093436.GI29387@submonkey.net> References: <200511040416.jA44GiAG061102@repoman.freebsd.org> <20051104071858.GT63539@ip.net.ua> <20051104093436.GI29387@submonkey.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Friday 04 November 2005 01:34 am, Ceri Davies wrote: > On Fri, Nov 04, 2005 at 09:18:58AM +0200, Ruslan Ermilov wrote: > > On Fri, Nov 04, 2005 at 04:16:44AM +0000, Warner Losh wrote: > > > imp 2005-11-04 04:16:44 UTC > > > > > > FreeBSD src repository > > > > > > Modified files: > > > usr.sbin/config configvers.h > > > Log: > > > since nocpu isn't used in the kernel config base, we don't need > > > to bump the version. Peter Wemm, John Baldwin and I hammered > > > this out after the last time I needlessly incremented the > > > version. > > > > > > Revision Changes Path > > > 1.38 +1 -1 src/usr.sbin/config/configvers.h > > > > Our users wanted to *use* the "nocpu" directive in their configs > > that are based on GENERIC (GENERIC is included), hence I bumped > > config(8). Otherwise, one day some would scream that "nocpu" > > doesn't work for them, because they're using the older version > > of config(8). > > I'd agree that the Makefiles don't need to be bumped, as they don't > actually require the new config, but config has new functionality and > the version should probably be bumped in the binary. The reason for configvers in the first place was that config(8) didn't fail for many error conditions at all. It would create a bogus kernel build environment even though a parser change caused hundreds of syntax errors. This was a real problem because 'make buildkernel' had become all the 'in' thing to do, and people were getting hosed because buildkernel/installkernel would quite happily install a bogus kernel after generating hundreds of lines of error messages. The victim would be none the wiser because the config output would be lost in the noise of the make output. config was originally written to be used interactively and the results read, not run from a make script. Over time, config was fixed to correctly return error codes for parser problems, and the syntax evolved so that errors were more likely to be detected. These days we generally don't need to bump configvers because syntax changes are correctly detected. configvers should only be bumped if they are NOT detected, or if the interface between /sys/conf/* and /usr/sbin/config has changed. Adding 'nocpu' does not require a configvers bump. An older config(8) will correctly fail with a syntax error if it is exposed to a newer 'nocpu' keyword. Needlessly bumping configvers is harmful because it disrupts users and developers. config should NOT be made to fail just because it is out of sync with 'nocpu' support whennocpu isn't used. That would be just silly. -- 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?200511041633.47778.peter>