From owner-cvs-src@FreeBSD.ORG Sat May 31 02:20:47 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9C99A37B401; Sat, 31 May 2003 02:20:47 -0700 (PDT) Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by mx1.FreeBSD.org (Postfix) with ESMTP id 73E7543F85; Sat, 31 May 2003 02:20:45 -0700 (PDT) (envelope-from bde@zeta.org.au) Received: from katana.zip.com.au (katana.zip.com.au [61.8.7.246]) by mailman.zeta.org.au (8.9.3p2/8.8.7) with ESMTP id TAA06052; Sat, 31 May 2003 19:20:41 +1000 Date: Sat, 31 May 2003 19:20:41 +1000 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: Ruslan Ermilov In-Reply-To: <20030530200512.GE60607@sunbay.com> Message-ID: <20030531191000.D7734@gamplex.bde.org> References: <200305301702.h4UH2bHi052360@repoman.freebsd.org> <20030530200512.GE60607@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: John Baldwin Subject: Re: cvs commit: src/sys/security/mac_mls mac_mls.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 31 May 2003 09:20:48 -0000 On Fri, 30 May 2003, Ruslan Ermilov wrote: > On Fri, May 30, 2003 at 02:25:10PM -0400, John Baldwin wrote: > > Can someone make module builds use the same warning flags and > > settings as kernel builds please? This same scenario has happened > > numerous times now. > > > This is not easy. For the kernel build, we have NORMAL_C and > NORMAL_C_NOWERROR (the former includes WERROR?= -Werror). > There are lot of files in sys/conf/files* that are marked with > "nowerror", and I'm sure they will break to compile as modules > with this patch, e.g., the acpi module has lot of files with > warnings: Not a lot, actually. There are 21 acpi files and 17 non-acpi files with nowerror. The acpi errors are less numerous than they used. About half of the other 17 are for unmaintained drivers and the other half are not needed or otherwise bogus. > %%% > Index: kern.mk > =================================================================== > RCS file: /home/ncvs/src/sys/conf/kern.mk,v > retrieving revision 1.34 > diff -u -r1.34 kern.mk > --- kern.mk 3 May 2003 18:05:56 -0000 1.34 > +++ kern.mk 30 May 2003 19:57:00 -0000 > @@ -9,6 +9,8 @@ > CWARNFLAGS?= -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes \ > -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual \ > -fformat-extensions -std=c99 > +WERROR?= -Werror > + > # > # The following flags are next up for working on: > # -W Style bug: separation of the -W options and comments by putting a non- -W option in the middile. The comment at the beginning of kern.mk was intended to apply to the whole file (the file should contain only warning options). This was broken in rev.1.11. Bruce