cvs commit: src/sys/contrib/dev/acpica acfreebsd.h
nate at root.org
Thu Aug 7 07:45:01 PDT 2003
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.
> /* 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.
More information about the cvs-src