[PATCH xinit] need includes for PRIO_PROCESS on OpenBSD too.

Mark Kettenis mark.kettenis at xs4all.nl
Mon Nov 1 03:41:24 PDT 2010


> Date: Mon, 1 Nov 2010 10:03:30 +0100
> From: Matthieu Herrb <matthieu.herrb at laas.fr>
> 
> Signed-off-by: Matthieu Herrb <matthieu.herrb at laas.fr>
> ---
>  xinit.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)

Actually the Linux man page suggests that even on Linux those two
header files should be included.  They might get picked up as a
side-effect of some other include, but it shouldn't hurt to just make
these includes unconditional isn't it?

> diff --git a/xinit.c b/xinit.c
> index 0d31637..dd4e0a2 100644
> --- a/xinit.c
> +++ b/xinit.c
> @@ -49,10 +49,10 @@ in this Software without prior written authorization from The Open Group.
>  #endif
>  
>  /* For PRIO_PROCESS and setpriority() */
> -#ifdef __DragonFly__
> +#if defined(__DragonFly__) || defined(__OpenBSD__)
>  #include <sys/time.h>
>  #include <sys/resource.h>
> -#endif /* __DragonFly__ */
> +#endif /* __DragonFly__ || __OpenBSD__ */
>  
>  #include <stdlib.h>


More information about the xorg-devel mailing list