From owner-cvs-all@FreeBSD.ORG Fri Oct 31 05:34:57 2003 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id E4D0716A4CE; Fri, 31 Oct 2003 05:34:57 -0800 (PST) Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8EE8A43FBF; Fri, 31 Oct 2003 05:34:55 -0800 (PST) (envelope-from bde@zeta.org.au) Received: from gamplex.bde.org (katana.zip.com.au [61.8.7.246]) by mailman.zeta.org.au (8.9.3p2/8.8.7) with ESMTP id AAA07704; Sat, 1 Nov 2003 00:34:52 +1100 From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: Ruslan Ermilov In-Reply-To: <20031031123129.GB777@sunbay.com> Message-ID: <20031101002134.Y1748@gamplex.bde.org> References: <200310310542.h9V5gr8v023991@repoman.freebsd.org> <20031031230028.K1418@gamplex.bde.org> <20031031123129.GB777@sunbay.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org cc: Peter Wemm Subject: Re: cvs commit: src/sys/sys cdefs.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Date: Fri, 31 Oct 2003 13:34:58 -0000 X-Original-Date: Sat, 1 Nov 2003 00:34:51 +1100 (EST) X-List-Received-Date: Fri, 31 Oct 2003 13:34:58 -0000 On Fri, 31 Oct 2003, Ruslan Ermilov wrote: > On Fri, Oct 31, 2003 at 11:15:40PM +1100, Bruce Evans wrote: > > ... > > While it is good to check for warnings in system headers, this shouldn't > > be forced on everyone. Applications that want to check for the non-error > > of using undefined identifiers in cpp expressions shouldn't enable > > warnings in system headers unless they want to get warnings if the headers > > use this standard C feature (and other warnings). Applications in the src > > tree won't do this. > > > What's the underlying issue here? I always thought that FreeBSD is different > from GNU is this respect -- we treat our system as the whole, so the piece > of code is as clean as its worst part; that may be a system header, too. > Remember the issue? Old versions of buildworld used -nostdinc hack and > -I... to ${WORLDTMP}/usr/include, and it caused WARNS to be generating > different warnings while in buildworld compared to a normal (no -nostdinc) > build. I'm against reverting the toplev.c change, but it also made the > sys.mk part redundant, so the latter may be reverted now. The system shouldn't have to satisfy cleanness standards of applications that are not maintained by FreeBSD, especially when those standards are bogus (e.g., compiling cleanly with -Wundef). Hopefull all the important corner cases in the system headers are tested by buildworld so that we don't need to enlist the application maintainers to find bugs in system headers. Bruce