From owner-cvs-all@FreeBSD.ORG  Mon May 30 08:16:41 2005
Return-Path: <owner-cvs-all@FreeBSD.ORG>
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 4A7D916A41C;
	Mon, 30 May 2005 08:16:41 +0000 (GMT)
	(envelope-from rwatson@FreeBSD.org)
Received: from cyrus.watson.org (cyrus.watson.org [204.156.12.53])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 0F81D43D1D;
	Mon, 30 May 2005 08:16:41 +0000 (GMT)
	(envelope-from rwatson@FreeBSD.org)
Received: from fledge.watson.org (fledge.watson.org [204.156.12.50])
	by cyrus.watson.org (Postfix) with ESMTP id AEEFF46B49;
	Mon, 30 May 2005 04:16:40 -0400 (EDT)
Date: Mon, 30 May 2005 09:17:05 +0100 (BST)
From: Robert Watson <rwatson@FreeBSD.org>
X-X-Sender: robert@fledge.watson.org
To: Scott Long <scottl@FreeBSD.org>
In-Reply-To: <200505300501.j4U51ikj070065@repoman.freebsd.org>
Message-ID: <20050530091556.A52379@fledge.watson.org>
References: <200505300501.j4U51ikj070065@repoman.freebsd.org>
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
Subject: Re: cvs commit: src/sys/dev/random probe.c src/sys/opencrypto 
 deflate.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 30 May 2005 08:16:41 -0000


On Mon, 30 May 2005, Scott Long wrote:

>  malloc.h relies on param.h for a definition of MAXCPU.  I guess that there is
>  other header pollution that makes this work right now, but it falls over when
>  doing a RELENG_5 -> HEAD upgrade.

Odd -- I added a nested include of param.h from malloc.h to catch this. 
While almost certainly a bad idea (your fix is better), if you had the 
latest revision of malloc.h, it shouldn't have broken the build...  Was 
your malloc.h slightly stale, or is there more to it than that?

I'll basck out my change to malloc.h now that the dependencies are fixed. 
Thanks!

Robert N M Watson