Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 19 May 2001 23:24:25 -0700
From:      Kris Kennaway <kris@obsecurity.org>
To:        Bruce Evans <bde@zeta.org.au>
Cc:        Kris Kennaway <kris@FreeBSD.org>, 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
Message-ID:  <20010519232425.A20670@xor.obsecurity.org>
In-Reply-To: <Pine.BSF.4.21.0105201548240.42146-100000@besplex.bde.org>; from bde@zeta.org.au on Sun, May 20, 2001 at 03:54:17PM %2B1000
References:  <200105192318.f4JNIMp86494@freefall.freebsd.org> <Pine.BSF.4.21.0105201548240.42146-100000@besplex.bde.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--u3/rZRmxL6MmkK24
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, May 20, 2001 at 03:54:17PM +1000, Bruce Evans wrote:
> On Sat, 19 May 2001, Kris Kennaway wrote:
>=20
> > kris        2001/05/19 16:18:21 PDT
> >=20
> >   Modified files:
> >     share/mk             Makefile bsd.kmod.mk bsd.lib.mk=20
> >                          bsd.prog.mk=20
> >   Added files:
> >     share/mk             bsd.sys.mk=20
> >   Log:
> >   Bring in the WARNS directive from NetBSD, localized in a new bsd.sys.=
mk
> >   to avoid polluting sys.mk.  This directive controls the addition of
> >   compiler warning flags to CFLAGS in a relatively compiler-neutral man=
ner.
> >  =20
> >   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
>=20
> Please don't clutter individual Makefiles with it.
>=20
> I think it would be more useful to fix all the bugs reported with the
> standard warning flags (-Wall for src/bin, lots for kernels, but none
> for everything else) first.  The output from `make -ss' should be null,
> but is currently almost 100K.  -Wall in individual Makefiles has had
> a negative effect in preventing new warnings (it causes so many warnings
> that people ignore them all).

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=3D0 on the remainder.

Kris
--u3/rZRmxL6MmkK24
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.5 (FreeBSD)
Comment: For info see http://www.gnupg.org

iD8DBQE7B2MZWry0BWjoQKURAiSfAJ41mHWZfjdzQmpRAsCAWq5GxB7iDQCfatDi
PQTpSmT+VN57huqgM4Zid1Q=
=A6JB
-----END PGP SIGNATURE-----

--u3/rZRmxL6MmkK24--

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010519232425.A20670>