From owner-cvs-all  Thu Feb  5 08:37:04 1998
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Received: (from majordom@localhost)
          by hub.freebsd.org (8.8.8/8.8.8) id IAA14735
          for cvs-all-outgoing; Thu, 5 Feb 1998 08:37:04 -0800 (PST)
          (envelope-from owner-cvs-all@FreeBSD.ORG)
Received: from ns1.yes.no (ns1.yes.no [195.119.24.10])
          by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id IAA14728
          for <cvs-committers@FreeBSD.ORG>; Thu, 5 Feb 1998 08:37:00 -0800 (PST)
          (envelope-from eivind@bitbox.follo.net)
Received: from bitbox.follo.net (bitbox.follo.net [194.198.43.36])
	by ns1.yes.no (8.8.7/8.8.7) with ESMTP id QAA27393;
	Thu, 5 Feb 1998 16:36:35 GMT
Received: (from eivind@localhost)
	by bitbox.follo.net (8.8.6/8.8.6) id RAA01740;
	Thu, 5 Feb 1998 17:36:34 +0100 (MET)
Message-ID: <19980205173634.06208@follo.net>
Date: Thu, 5 Feb 1998 17:36:34 +0100
From: Eivind Eklund <eivind@yes.no>
To: Bruce Evans <bde@zeta.org.au>
Cc: eivind@yes.no, wollman@khavrinen.lcs.mit.edu, cvs-committers@FreeBSD.ORG
Subject: Re: cvs commit: src/sys/conf options src/sys/gnu/ext2fs ext2_alloc.c ext2_balloc.c ext2_inode.c ext2_lookup.c ext2_subr.c ext2_vnops.c src/sys/i386/i386 busdma_machdep.c machdep.c pmap.c swtch.s trap.c vm86.c vm_machdep.c src/sys/i386/ibcs2 ibcs2_sysi86.
References: <199802050954.UAA31717@godzilla.zeta.org.au>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-Mailer: Mutt 0.88e
In-Reply-To: <199802050954.UAA31717@godzilla.zeta.org.au>; from Bruce Evans on Thu, Feb 05, 1998 at 08:54:38PM +1100
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
X-To-Unsubscribe: mail to majordomo@FreeBSD.org "unsubscribe cvs-all"

On Thu, Feb 05, 1998 at 08:54:38PM +1100, Bruce Evans wrote:
> >>[DIAGNOSTIC]
> 
> >Well, the way I've always used it was to turn it on for just those
> >specific modules I was debugging.  That keeps the code perturbation
> 
> That should still work, since if DIAGNOSTIC is not configured, then
> opt_diagnostic.h will not define DIAGNOSTIC, so you can still use
> `make COPTFLAGS=-DDIAGNOSTIC'.  `make COPTFLAGS=-DDIAGNOSTIC=0' won't
> work to turn off DIAGNOSTIC if it is configured, but it is already
> broken, since COPTFLAGS and COPTS are almost perfectly misplaced at
> the beginning and near the end of CFLAGS respectively, so COPTFLAGS
> can't be used to override COPTS.
> 
> Similarly for DEBUG.

So I can throw DEBUG in opt_global.h (or potentially opt_debug.h, but
I think DEBUGs scope change too often for that to be useful)?  Then
we'll have no old-style options as default - yi-ha! :-)

Eivind.