From owner-cvs-all Sun May 20 4: 3:28 2001 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 4D24D37B424; Sun, 20 May 2001 04:03:21 -0700 (PDT) (envelope-from bde@zeta.org.au) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id VAA00012; Sun, 20 May 2001 21:03:10 +1000 Date: Sun, 20 May 2001 21:01:44 +1000 (EST) From: Bruce Evans X-Sender: bde@besplex.bde.org To: Kris Kennaway Cc: Kris Kennaway , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/share/mk bsd.sys.mk Makefile bsd.kmod.mk bsd.lib.mk bsd.prog.mk In-Reply-To: <20010519232425.A20670@xor.obsecurity.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sat, 19 May 2001, Kris Kennaway wrote: > On Sun, May 20, 2001 at 03:54:17PM +1000, Bruce Evans wrote: > > On Sat, 19 May 2001, Kris Kennaway wrote: > > > The idea is that WARNS can be set in Makefile.inc or in individual > > > Makefiles as they become clean, to prevent the introduction of new > > > warnings in the code. -Werror is added by default > > > > Please don't clutter individual Makefiles with it. > > I'm only adding WARNS once I've fixed all warnings on alpha and i386. Since > it has the effect of adding -Werror, it's kind of hard for people to ignore > it when they introduce a new one :-) For people doing a new architecture > port or playing with weird compilation options, NO_WERROR or NO_WARNS is > available. > > Once we get all (or most) of the subdirectories building with WARNS, we can > shift it to ../Makefile.inc and set WARNS=0 on the remainder. Just don't set it for everyone until you (or someone) has fixed all (or most) of the subdirectories. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message