Date: Fri, 18 Sep 2020 12:19:42 -0500 From: Kyle Evans <kevans@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r365887 - in head: . bin gnu/lib/libregex gnu/usr.bin/diff3 gnu/usr.bin/gdb gnu/usr.bin/grep lib lib/clang lib/libarchive/tests lib/libc/tests/regex lib/libcasper/services/cap_dns/tests... Message-ID: <CACNAnaEni7qaQY72QMnnS1PX25cn2X4mM%2Beva%2Bknor%2BncCgEfQ@mail.gmail.com> In-Reply-To: <202009181717.08IHHkSq047581@repo.freebsd.org> References: <202009181717.08IHHkSq047581@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Sep 18, 2020 at 12:18 PM Kyle Evans <kevans@freebsd.org> wrote: > > Author: kevans > Date: Fri Sep 18 17:17:46 2020 > New Revision: 365887 > URL: https://svnweb.freebsd.org/changeset/base/365887 > > Log: > build: provide a default WARNS for all in-tree builds > > The current default is provided in various Makefile.inc in some top-level > directories and covers a good portion of the tree, but doesn't cover parts > of the build a little deeper (e.g. libcasper). > > Provide a default in src.sys.mk and set WARNS to it in bsd.sys.mk if that > variable is defined. This lets us relatively cleanly provide a default WARNS > no matter where you're building in the src tree without breaking things > outside of the tree. > > Crunchgen has been updated as a bootstrap tool to work on this change > because it needs r365605 at a minimum to succeed. The cleanup necessary to > successfully walk over this change on WITHOUT_CLEAN builds has been added. > > There is a supplemental project to this to list all of the warnings that are > encountered when the environment has WARNS=6 NO_WERROR=yes: > https://warns.kevans.dev -- this project will hopefully eventually go away > in favor of CI doing a much better job than it. > > Reviewed by: emaste, brooks, ngie (all earlier version) > Reviewed by: emaste, arichardson (depend-cleanup.sh change) > Differential Revision: https://reviews.freebsd.org/D26455 > *sigh* Differential Revision: https://reviews.freebsd.org/D26397 D26455 was just the depend-cleanup part.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaEni7qaQY72QMnnS1PX25cn2X4mM%2Beva%2Bknor%2BncCgEfQ>