From owner-cvs-all  Sun Oct 13  4:58:15 2002
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 1542C37B404; Sun, 13 Oct 2002 04:58:14 -0700 (PDT)
Received: from espresso.q9media.com (espresso.q9media.com [65.39.129.122])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 9B96143E91; Sun, 13 Oct 2002 04:58:13 -0700 (PDT)
	(envelope-from mike@espresso.q9media.com)
Received: by espresso.q9media.com (Postfix, from userid 1002)
	id 1DB739C0A; Sun, 13 Oct 2002 07:50:39 -0400 (EDT)
Date: Sun, 13 Oct 2002 07:50:39 -0400
From: Mike Barcroft <mike@FreeBSD.org>
To: Marcel Moolenaar <marcel@xcllnt.net>
Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/sys signal.h
Message-ID: <20021013075039.A22315@espresso.q9media.com>
References: <200210130033.g9D0XqWR072184@freefall.freebsd.org> <20021013033514.GA80994@dhcp01.pn.xcllnt.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20021013033514.GA80994@dhcp01.pn.xcllnt.net>; from marcel@xcllnt.net on Sat, Oct 12, 2002 at 08:35:14PM -0700
Organization: The FreeBSD Project
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

Marcel Moolenaar <marcel@xcllnt.net> writes:
> On Sat, Oct 12, 2002 at 05:33:52PM -0700, Mike Barcroft wrote:
> > mike        2002/10/12 17:33:52 PDT
> [snip]
> >   Revision  Changes    Path
> >   1.34      +70 -49    src/sys/sys/signal.h
> 
> Mike,
> 
> This commit breaks a kernel build for kern/kern_mib.c. The problem is
> that kern_mib.c uses _KPOSIX_VERSION that was indirectly pulled in
> by sys/signal.h by means of the inclusion of sys/_posix.h (I think).
> We now don't have any consumers of _posix.h...

Oops, I forgot that LINT explicitly defines a _KPOSIX_VERSION, so I
should have tested it with GENERIC too.  I've committed a fix.  The
include path was even more complicated than this since <sys/proc.h>
was including <sys/signal.h> which was including <sys/_posix.h>.

Best regards,
Mike Barcroft

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message