Thanks for the patch. On Sun, Jun 13, 2010 at 10:15:17 +0200, Cyril Brulebois wrote:
> --- a/src/gallium/auxiliary/os/os_time.c
> +++ b/src/gallium/auxiliary/os/os_time.c
> @@ -37,7 +37,7 @@
>
> #if !defined(PIPE_OS_EMBEDDED)
>
> -#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_BSD) ||
> defined(PIPE_OS_SOLARIS) || defined(PIPE_OS_APPLE) || defined(PIPE_OS_HAIKU)
> +#if defined(PIPE_OS_LINUX) || defined(PIPE_OS_BSD) ||
> defined(PIPE_OS_SOLARIS) || defined(PIPE_OS_APPLE) || defined(PIPE_OS_HAIKU)
> || defined(__GLIBC__)
> # include <sys/time.h> /* timeval */
> #elif defined(PIPE_SUBSYSTEM_WINDOWS_DISPLAY)
> # include <windows.h>
Should probably change src/gallium/include/pipe/p_config.h to set the
right PIPE_* defines, and check everywhere they're used that we're going
with the right path.
03:13 < jcristau> looks like there are 2 places where PIPE_OS_BSD and
PIPE_OS_LINUX do different things. rtasm/rtasm_execmem.c we
want the linux path, and util/u_cpu_detect.c we want the bsd
path.. :)
I didn't check the PIPE_OS_FREEBSD uses.
Cheers,
Julien
signature.asc
Description: Digital signature
