From owner-cvs-all@FreeBSD.ORG Wed Aug 16 12:48:04 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C98C516A4DD; Wed, 16 Aug 2006 12:48:04 +0000 (UTC) (envelope-from bde@zeta.org.au) Received: from mailout2.pacific.net.au (mailout2.pacific.net.au [61.8.0.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 5966043D70; Wed, 16 Aug 2006 12:48:00 +0000 (GMT) (envelope-from bde@zeta.org.au) Received: from mailproxy2.pacific.net.au (mailproxy2.pacific.net.au [61.8.2.163]) by mailout2.pacific.net.au (Postfix) with ESMTP id F0E086E950; Wed, 16 Aug 2006 22:47:58 +1000 (EST) Received: from katana.zip.com.au (katana.zip.com.au [61.8.7.246]) by mailproxy2.pacific.net.au (8.13.4/8.13.4/Debian-3sarge1) with ESMTP id k7GClpMF027713; Wed, 16 Aug 2006 22:47:57 +1000 Date: Wed, 16 Aug 2006 22:47:51 +1000 (EST) From: Bruce Evans X-X-Sender: bde@delplex.bde.org To: Ruslan Ermilov In-Reply-To: <20060815083441.GD67570@rambler-co.ru> Message-ID: <20060816221914.H37557@delplex.bde.org> References: <200608141328.k7EDSrRB084019@repoman.freebsd.org> <20060815085001.S32503@delplex.bde.org> <20060815083441.GD67570@rambler-co.ru> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, Dag-Erling Smorgrav Subject: Re: cvs commit: src/sys/conf kern.post.mk kmod.mk src/sys/modules/if_ef Makefile src/sys/modules/if_ppp Makefile src/sys/modules/netgraph/mppc Makefile src/sys/modules/sound/sound Makefile X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Aug 2006 12:48:04 -0000 On Tue, 15 Aug 2006, Ruslan Ermilov wrote: > On Tue, Aug 15, 2006 at 09:40:36AM +1000, Bruce Evans wrote: >> On Mon, 14 Aug 2006, Dag-Erling Smorgrav wrote: >>> Revision Changes Path >>> 1.96 +1 -1 src/sys/conf/kern.post.mk >>> 1.210 +2 -2 src/sys/conf/kmod.mk >>> 1.14 +4 -4 src/sys/modules/if_ef/Makefile >>> 1.36 +3 -3 src/sys/modules/if_ppp/Makefile >>> 1.14 +1 -1 src/sys/modules/netgraph/mppc/Makefile >>> 1.19 +1 -1 src/sys/modules/sound/sound/Makefile >> >> I use echo -n for one of these. "echo" is less cryptic than ">", and >> is probably faster since /bin/echo is smaller than /bin/sh and is used >> enough to keep it cached. >> > make(1) will still use sh(1) when it sees ">", so what's the point? Oops. For some reason I thought make handled redirection internally, and ktrace didn't seem to show sh. >> In sound/Makefile, the cleanup should be to remove the touching (now >> emptying) since the touching used to be just worse than the default of >> emptying and is now just the same as the default (but takes extra code). >> > No. It has a preceding comment explaining why it's needed on sparc64: > to always build sound.ko without isadma dependency; see rev. 1.16 by > marius@. Hmm, that's ugly. 1.16 seems to have the opposite effect in the non-sparc64 case -- it forces isa for sound.ko even in the KERNBUILDDIR case with isa not in the kernel. All targets for opt_foo.h in module makefiles similarly force the kernel options to not be used. However, it is a bug to ever use kernel options in modules :-). Bruce