From owner-svn-src-head@FreeBSD.ORG Wed Jul 1 13:15:36 2009 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0CDE41065672; Wed, 1 Jul 2009 13:15:36 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail03.syd.optusnet.com.au (mail03.syd.optusnet.com.au [211.29.132.184]) by mx1.freebsd.org (Postfix) with ESMTP id 967AF8FC08; Wed, 1 Jul 2009 13:15:35 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from c122-106-161-96.carlnfd1.nsw.optusnet.com.au (c122-106-161-96.carlnfd1.nsw.optusnet.com.au [122.106.161.96]) by mail03.syd.optusnet.com.au (8.13.1/8.13.1) with ESMTP id n61DFL0G018412 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 1 Jul 2009 23:15:33 +1000 Date: Wed, 1 Jul 2009 23:15:22 +1000 (EST) From: Bruce Evans X-X-Sender: bde@delplex.bde.org To: Stanislav Sedov In-Reply-To: <20090701124825.af0d205e.stas@FreeBSD.org> Message-ID: <20090701223719.L39215@delplex.bde.org> References: <200906302209.n5UM9lBM093495@svn.freebsd.org> <20090701124825.af0d205e.stas@FreeBSD.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Martin Blapp , src-committers@freebsd.org Subject: Re: svn commit: r195213 - head/usr.sbin/wake X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 01 Jul 2009 13:15:36 -0000 On Wed, 1 Jul 2009, Stanislav Sedov wrote: > On Tue, 30 Jun 2009 22:09:47 +0000 (UTC) > Martin Blapp mentioned: >> Log: >> Compiles without higher warn level >> >> MAN= wake.8 >> >> -WARNS?= 2 >> - > > I believe you wanted something like WARNS?=6 probably? Now you disabled > any warnings-related compiler flags entirely. WARNS should only be set in near-top-level Makefile.inc's. Setting it elsewhere is either a style bug to increase the warning level or a hack to decrease the warning level. However, setting of WARNS in near-top- level Makefile.inc's stalled several years ago at the same time as its excessive setting in individual Makefiles. It is still not set at all in usr.sbin/Makefile.inc, while in usr.bin its setting is commented out "temporarily". Only the relatively simple bin directory has been handled well -- bin/Makefile.inc and most files under it started uniformly at WARNS=2, and bin/Makefile.inc has now reached WARNS=6, with only the following bugs and hacks in leaf Makefiles: % ./rmail/Makefile:WARNS?= 0 Apparently, sendmail isn't written in C yet. % ./cat/Makefile:WARNS?= 6 Bogus override of the default to the same value. cat compiled happily with the default WARNS of 6 for 6 years before this bug was committed in 2009, after being fixed in 2001 to not set WARNS, so that it automatically picked up the change of the default from 2 to 6 in 2003. % ./csh/Makefile:WARNS?= 0 Csh was never written in C :-). % ./ed/Makefile:WARNS?= 2 % ./pax/Makefile:WARNS?= 3 % ./pkill/Makefile:WARNS?= 5 % ./sh/Makefile:WARNS?= 2 Assorted hacks. % ./uuidgen/Makefile:WARNS?= 6 Another bogus override of the default. In usr.sbin, WARNS has never been turned on globally, and slightly more than half of the makefiles don't set it locally either. Bruce