From owner-freebsd-arch@FreeBSD.ORG  Mon Mar 22 19:08:51 2010
Return-Path: <owner-freebsd-arch@FreeBSD.ORG>
Delivered-To: freebsd-arch@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 97575106566B;
	Mon, 22 Mar 2010 19:08:51 +0000 (UTC) (envelope-from imp@bsdimp.com)
Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85])
	by mx1.freebsd.org (Postfix) with ESMTP id 576838FC25;
	Mon, 22 Mar 2010 19:08:51 +0000 (UTC)
Received: from localhost (localhost [127.0.0.1])
	by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id o2MJ5B60028317;
	Mon, 22 Mar 2010 13:05:11 -0600 (MDT) (envelope-from imp@bsdimp.com)
Date: Mon, 22 Mar 2010 13:05:12 -0600 (MDT)
Message-Id: <20100322.130512.864843819464264610.imp@bsdimp.com>
To: Alexander@Leidinger.net
From: "M. Warner Losh" <imp@bsdimp.com>
In-Reply-To: <20100322.125937.278730673160410010.imp@bsdimp.com>
References: <201003220941.10525.jhb@freebsd.org>
	<20100322172104.14234yawbsev0sw8@webmail.leidinger.net>
	<20100322.125937.278730673160410010.imp@bsdimp.com>
X-Mailer: Mew version 6.3 on Emacs 22.3 / Mule 5.0 (SAKAKI)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Cc: rwatson@freebsd.org, freebsd-arch@freebsd.org
Subject: Re: CTF patch for testing/review
X-BeenThere: freebsd-arch@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussion related to FreeBSD architecture <freebsd-arch.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arch>
List-Post: <mailto:freebsd-arch@freebsd.org>
List-Help: <mailto:freebsd-arch-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 22 Mar 2010 19:08:51 -0000

In message: <20100322.125937.278730673160410010.imp@bsdimp.com>
            M. Warner Losh <imp@bsdimp.com> writes:
: In message: <20100322172104.14234yawbsev0sw8@webmail.leidinger.net>
:             Alexander Leidinger <Alexander@Leidinger.net> writes:
: : Normally we use MK_xxx for things which are opt-in/opt-out. What about
: : using MK_xxx instead of ENABLE_CTF? If people are in favour of MK_xxx,
: : what should the xxx part look like?
: 
: Normally we *TEST* MK_XXX for things which are opt-in/opt-out and
: require the user to say WITH_XXX or WITHOUT_XXX if they don't like the
: default (or want to ensure they get option XXX, even if we turn it off
: by default in the future).  The default then gets encoded in
: bsd.own.mk, and permeates the FreeBSD build system since we include
: that everywhere, directly or indirectly.
: 
: The problem is that bsd.own.mk is not included in sys.mk, nor should
: it be.  That's why we have the hacky combination of WITH_CTF and
: NO_CTF that's there today.
: 
: : Is bsd.kern.mk included in module builds too?
: 
: Yes.

One last thing I should have said was that the patch that was posted
earlier in the thread looked ok, and likely couldn't be made
significantly better due to the bsd.own.mk issue.

Warner

P.S.  Sorry for the lame followup to my own post.