[Buildroot] [PATCH v2, 1/1] package/weston: fix build with headers < 4.4

Fabrice Fontaine fontaine.fabrice at gmail.com
Mon Jun 10 10:33:26 UTC 2019


weston includes input-event-codes.h since version 5.0.91 and
https://github.com/wayland-project/weston/commit/6e229ca26381bc8191fd9af1e439c311da709aff

input-event-codes.h is available only since kernel 4.4 and
https://github.com/torvalds/linux/commit/f902dd893427eade90f7eaf858e5ff8b150a5a12

To fix this build failure, check for the availability of this header
before including it and fallback to linux/input.h (for EV_KEY) if needed

It should be noted that we could just drop linux/input.h as it is
already pulled in from libevdev/libevdev.h

Fixes:
 - http://autobuild.buildroot.org/results/210c2759900f15ea0030d088f6f45cd8bb199b29

Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
---
Changes v1 -> v2 (after review of Yann E. Morin):
 - Add a patch to drop include on input-event-codes.h instead of
   requiring headers >= 4.4 

 .../0002-Fix-build-with-kernel-4.4.patch      | 59 +++++++++++++++++++
 1 file changed, 59 insertions(+)
 create mode 100644 package/weston/0002-Fix-build-with-kernel-4.4.patch

diff --git a/package/weston/0002-Fix-build-with-kernel-4.4.patch b/package/weston/0002-Fix-build-with-kernel-4.4.patch
new file mode 100644
index 0000000000..05b499794e
--- /dev/null
+++ b/package/weston/0002-Fix-build-with-kernel-4.4.patch
@@ -0,0 +1,59 @@
+From f2459cdf188a720384099b82ec40cf1915299552 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice at gmail.com>
+Date: Mon, 10 Jun 2019 12:02:03 +0200
+Subject: [PATCH] Fix build with kernel < 4.4
+
+weston includes input-event-codes.h since version 5.0.91 and
+https://github.com/wayland-project/weston/commit/6e229ca26381bc8191fd9af1e439c311da709aff
+
+input-event-codes.h is available only since kernel 4.4 and
+https://github.com/torvalds/linux/commit/f902dd893427eade90f7eaf858e5ff8b150a5a12
+
+To fix this build failure, check for the availability of this header
+before including it and fallback to linux/input.h (for EV_KEY) if needed
+
+It should be noted that we could just drop linux/input.h as it is
+already pulled in from libevdev/libevdev.h
+
+Fixes:
+ - http://autobuild.buildroot.org/results/210c2759900f15ea0030d088f6f45cd8bb199b29
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
+[Upstream status:
+https://gitlab.freedesktop.org/wayland/weston/merge_requests/184]
+---
+ compositor/main.c | 4 ++++
+ meson.build       | 1 +
+ 2 files changed, 5 insertions(+)
+
+diff --git a/compositor/main.c b/compositor/main.c
+index 945f99ae..dfac03f6 100644
+--- a/compositor/main.c
++++ b/compositor/main.c
+@@ -42,7 +42,11 @@
+ #include <sys/socket.h>
+ #include <libinput.h>
+ #include <libevdev/libevdev.h>
++#ifdef HAVE_LINUX_INPUT_EVENT_CODES_H
+ #include <linux/input-event-codes.h>
++#else
++#include <linux/input.h>
++#endif
+ #include <sys/time.h>
+ #include <linux/limits.h>
+ 
+diff --git a/meson.build b/meson.build
+index 55190028..a728d903 100644
+--- a/meson.build
++++ b/meson.build
+@@ -85,6 +85,7 @@ foreach func : optional_libc_funcs
+ endforeach
+ 
+ optional_system_headers = [
++	'linux/input-event-codes.h',
+ 	'linux/sync_file.h'
+ ]
+ foreach hdr : optional_system_headers
+-- 
+2.20.1
+
-- 
2.20.1



More information about the buildroot mailing list