From owner-cvs-all@FreeBSD.ORG Thu Aug 7 07:45:00 2003 Return-Path: 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 70BA737B404 for ; Thu, 7 Aug 2003 07:45:00 -0700 (PDT) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 02E0E43FE0 for ; Thu, 7 Aug 2003 07:44:58 -0700 (PDT) (envelope-from nate@rootlabs.com) Received: (qmail 75854 invoked by uid 1000); 7 Aug 2003 14:44:59 -0000 Date: Thu, 7 Aug 2003 07:44:59 -0700 (PDT) From: Nate Lawson To: "Daniel C. Sobral" In-Reply-To: <3F325F9A.8020906@tcoip.com.br> Message-ID: <20030807073437.X75796@root.org> References: <200308070446.h774kHAq037869@repoman.freebsd.org> <3F325F9A.8020906@tcoip.com.br> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/contrib/dev/acpica acfreebsd.h X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 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: Thu, 07 Aug 2003 14:45:01 -0000 On Thu, 7 Aug 2003, Daniel C. Sobral wrote: > This commit broke world on loader. Thanks, already fixed. > The problem is sys/boot/i386/libi386/biosacpi.c, which includes this > file since your rev 1.5 commit. :-) No, biosacpi.c does not include ctype.h. The commit you reference added acconfig.h, which is just a set of defines. > Specifically, stand.h, which is included by anything that uses libstand > functions (such as is the case with biosacpi.c), is mutually exclusive > with ctype.h. > > Specifically: > > /* Not building kernel code, so use libc */ > > is false on both accounts for loader. :-) Loader is not kernel, alright, > but neither does it use libc. > > Since I'm not familiar with how, exactly, is acfreebsd.h used, I can't > offer more advise. :-) > > Tricky, eh? :-) Since biosacpi is so small and doesn't use any Acpi* functions, the ACPI_USE_STANDARD_HEADERS define doesn't affect it in any way. The way it works is that acfreebsd.h sets up some compatibility includes/defines so that acpica builds ok. It's needed to make the structs used by biosacpi.c (Xsdt, Rsdt) work. > Shouldn't ctype.h be a required include to anything including acfreebsd.h? That could be one way. I posted the problem to current@ because I don't want to increase header pollution. -Nate