[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