From 797de283c7ae9b99e4d2366cf142b2bc8b2f2a84 Mon Sep 17 00:00:00 2001 From: Peter Korsgaard Date: Mon, 11 Dec 2023 17:45:29 +0100 Subject: [PATCH] package/xwayland: libdrm is required, not optional Fixes: http://autobuild.buildroot.net/results/476/47665d417dbae76bf27e805a5bcb1d8d6ab1f445/ xwayland unconditionally includes xf86drm.h, so libdrm is required and not optional: grep -rs xf86drm.h glamor/glamor_egl.c:#include hw/xwayland/xwayland-glamor.h:#include hw/xwayland/xwayland-glamor-eglstream.c:#include hw/xwayland/xwayland-window.h:#include hw/xwayland/xwayland-drm-lease.c:#include hw/xwayland/xwayland-glamor-gbm.c:#include Signed-off-by: Peter Korsgaard --- package/x11r7/xwayland/Config.in | 1 + package/x11r7/xwayland/xwayland.mk | 13 ++++--------- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/package/x11r7/xwayland/Config.in b/package/x11r7/xwayland/Config.in index 241ab73ad3..9ac1928f51 100644 --- a/package/x11r7/xwayland/Config.in +++ b/package/x11r7/xwayland/Config.in @@ -8,6 +8,7 @@ config BR2_PACKAGE_XWAYLAND # them, otherwise, use the small libsha1 library. depends on !BR2_RISCV_32 # xlib_libxshmfence depends on BR2_TOOLCHAIN_HAS_SYNC_4 # xlib_libxshmfence + select BR2_PACKAGE_LIBDRM select BR2_PACKAGE_LIBSHA1 if (!BR2_PACKAGE_OPENSSL && !BR2_PACKAGE_LIBGCRYPT) select BR2_PACKAGE_PIXMAN select BR2_PACKAGE_WAYLAND diff --git a/package/x11r7/xwayland/xwayland.mk b/package/x11r7/xwayland/xwayland.mk index 8eac602042..b0bff19be5 100644 --- a/package/x11r7/xwayland/xwayland.mk +++ b/package/x11r7/xwayland/xwayland.mk @@ -12,6 +12,7 @@ XWAYLAND_LICENSE_FILES = COPYING XWAYLAND_CPE_ID_VENDOR = x.org XWAYLAND_INSTALL_STAGING = YES XWAYLAND_DEPENDENCIES = \ + libdrm \ pixman \ wayland \ wayland-protocols \ @@ -23,15 +24,16 @@ XWAYLAND_DEPENDENCIES = \ xlib_xtrans \ xorgproto XWAYLAND_CONF_OPTS = \ + -Ddri3=true \ -Dxwayland_eglstream=false \ -Dxvfb=false \ -Ddefault_font_path=/usr/share/fonts/X11/ \ -Ddtrace=false \ -Ddocs=false -ifeq ($(BR2_PACKAGE_LIBDRM)$(BR2_PACKAGE_LIBEPOXY),yy) +ifeq ($(BR2_PACKAGE_LIBEPOXY),y) XWAYLAND_CONF_OPTS += -Dglamor=true -XWAYLAND_DEPENDENCIES += libdrm libepoxy +XWAYLAND_DEPENDENCIES += libepoxy else XWAYLAND_CONF_OPTS += -Dglamor=false endif @@ -79,13 +81,6 @@ XWAYLAND_CONF_OPTS += -Dsha1=libsha1 XWAYLAND_DEPENDENCIES += libsha1 endif -ifeq ($(BR2_PACKAGE_LIBDRM),y) -XWAYLAND_CONF_OPTS += -Ddri3=true -XWAYLAND_DEPENDENCIES += libdrm -else -XWAYLAND_CONF_OPTS += -Ddri3=false -endif - ifeq ($(BR2_PACKAGE_LIBUNWIND),y) XWAYLAND_CONF_OPTS += -Dlibunwind=true XWAYLAND_DEPENDENCIES += libunwind