[Mesa-stable] [Mesa-dev] [PATCH] loader: include <sys/stat.h> for non-sysfs builds

Ian Romanick idr at freedesktop.org
Wed Mar 11 13:50:36 PDT 2015


On 03/11/2015 12:12 PM, Emil Velikov wrote:
> Required by fstat(), otherwise we'll error out due to implicit function
> declaration.

Alternate suggestion... include unistd.h unconditionally.  I see that's
already in the #ifdef HAVE_LIBUDEV path.  Does that work too?

> Cc: "10.4 10.5" <mesa-stable at lists.freedesktop.org>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89530
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  src/loader/loader.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/loader/loader.c b/src/loader/loader.c
> index 94c993a..638ebf2 100644
> --- a/src/loader/loader.c
> +++ b/src/loader/loader.c
> @@ -64,6 +64,7 @@
>   *    Rob Clark <robclark at freedesktop.org>
>   */
>  
> +#include <sys/stat.h>
>  #include <stdarg.h>
>  #include <stdio.h>
>  #include <string.h>
> @@ -80,7 +81,6 @@
>  #endif
>  #endif
>  #ifdef HAVE_SYSFS
> -#include <sys/stat.h>
>  #include <sys/types.h>
>  #endif
>  #include "loader.h"
> 



More information about the mesa-stable mailing list