build: depend on xproto explicitly

Currently it's required by xkbcommon. However, we will need it for other
backends as well, even with xkbcommon support disabled.

xproto consisted only of static header files. We use the XK_ keysym
definitions. It is therefore only required during compilation and we
shouldn't needlessly duplicate it ourselves.

Signed-off-by: Ran Benita <ran234@gmail.com>
Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>

Makefile.am

@@ -70,6 +70,7 @@ libkmscon_core_la_CPPFLAGS = \
 	$(PANGO_CFLAGS) \
 	$(UDEV_CFLAGS) \
 	$(FREETYPE2_CFLAGS) \
+	$(XPROTO_CFLAGS) \
 	$(XKBCOMMON_CFLAGS) \
 	$(GLIB_CFLAGS)
 libkmscon_core_la_LIBADD = \
@@ -80,6 +81,7 @@ libkmscon_core_la_LIBADD = \
 	$(PANGO_LIBS) \
 	$(UDEV_LIBS) \
 	$(FREETYPE2_LIBS) \
+	$(XPROTO_LIBS) \
 	$(XKBCOMMON_LIBS) \
 	$(GLIB_LIBS)
 

README

@@ -9,6 +9,7 @@ console.
     - mesa: providing an OpenGL implementation (must be compiled with EGL, gbm
             and GL libraries)
     - udev: providing input device hotplug
+    - xproto (build time dependency): definition of key symbols
     - libxkbcommon: keyboard handling
     - glib: only for Unicode handling
     - One of:

configure.ac

@@ -56,6 +56,10 @@ PKG_CHECK_MODULES([UDEV], [libudev])
 AC_SUBST(UDEV_CFLAGS)
 AC_SUBST(UDEV_LIBS)
 
+PKG_CHECK_MODULES([XPROTO], [xproto])
+AC_SUBST(XPROTO_CFLAGS)
+AC_SUBST(XPROTO_LIBS)
+
 PKG_CHECK_MODULES([XKBCOMMON], [xkbcommon])
 AC_SUBST(XKBCOMMON_CFLAGS)
 AC_SUBST(XKBCOMMON_LIBS)