[Intel-gfx] [PATCH] backlight: Fix compile error

Chris Wilson chris at chris-wilson.co.uk
Wed Jun 18 14:51:07 CEST 2014


On Wed, Jun 18, 2014 at 02:41:56PM +0200, Hans de Goede wrote:
> /usr/include/xorg/os.h around line 579 reads:
> 
> extern _X_EXPORT char *
> strndup(const char *str, size_t n);
> 
> However strndup is already defined by glibc, and this redefine causes a
> compile error.
> 
> This gets triggered because backlight.c does:
> 
> Without first doing:
> 
> Causing HAVE_STRNDUP to not be defined.
> 
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
>  src/backlight.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/backlight.c b/src/backlight.c
> index d5b10a0..62241ab 100644
> --- a/src/backlight.c
> +++ b/src/backlight.c
> @@ -42,6 +42,7 @@
>  #include <fcntl.h>
>  #include <unistd.h>
>  #include <dirent.h>
> +#include <xorg-server.h>
>  #include <xf86.h>
>  #include <pciaccess.h>

Does ordering it as

diff --git a/src/backlight.c b/src/backlight.c
index d5b10a0..24eb11c 100644
--- a/src/backlight.c
+++ b/src/backlight.c
@@ -29,6 +29,9 @@
 #include "config.h"
 #endif
 
+#include <xorg-server.h>
+#include <xf86.h>
+
 #include <sys/types.h>
 #include <sys/wait.h>
 #include <sys/stat.h>
@@ -42,7 +45,6 @@
 #include <fcntl.h>
 #include <unistd.h>
 #include <dirent.h>
-#include <xf86.h>
 #include <pciaccess.h>
 
 #include "backlight.h"

make more sense?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list