[Buildroot] [PATCH 22/30] package/matchbox: forcibly disable support for compositing
Arnout Vandecappelle
arnout at mind.be
Fri May 1 20:31:23 UTC 2015
On 22/04/15 20:09, Yann E. MORIN wrote:
> Building with the experimental support for compositing is broken:
>
> /home/ymorin/dev/buildroot/O/host/usr/bin/arm-linux-gnueabihf-gcc
> -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64
> -Os -o matchbox-window-manager main.o wm.o base_client.o main_client.o
> toolbar_client.o toolbar_client_alt.o dockbar_client.o dialog_client.o
> select_client.o desktop_client.o ewmh.o misc.o client_common.o keys.o
> list.o stack.o composite-engine.o session.o mbtheme.o xml.o
> -lmb -lX11
> -L/home/ymorin/dev/buildroot/O/host/usr/arm-buildroot-linux-gnueabihf/sysroot/usr/lib
> -lXext -lXcomposite -lXdamage -lXfixes -lXrender
> -L/home/ymorin/dev/buildroot/O/host/usr/arm-buildroot-linux-gnueabihf/sysroot/usr/lib
> -lX11 -lexpat
> -L/home/ymorin/dev/buildroot/O/host/usr/arm-buildroot-linux-gnueabihf/sysroot/usr/lib
> -lXfixes
> composite-engine.o: In function `gaussian':
> /home/ymorin/dev/buildroot/O/build/matchbox-1.2/src/composite-engine.c:65:
> undefined reference to `exp'
>
> That's because it forgets to link with -lm.
>
> However, adding "MATCHBOX_CONF_ENV = -lm" does not solve the issue, as
> it still does not link with -lm at all.
>
> Furthermore, the package does not autoreconf, since it is missing the
> gconf m4 macros, and Buildroot does not have a package for gconf, hence
> we can not even patch Makefile.am and autoreconf.
>
> Patching Makefile.in (in addition to .am) could be a solution, but
> support for compositing is explicitly marked as being experimental.
>
> So, just forcibly disable it altogether.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
It kind of makes the previous patch redundant, but there you go...
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
> ---
> package/matchbox/matchbox.mk | 13 +------------
> 1 file changed, 1 insertion(+), 12 deletions(-)
>
> diff --git a/package/matchbox/matchbox.mk b/package/matchbox/matchbox.mk
> index be94414..4ec3fb5 100644
> --- a/package/matchbox/matchbox.mk
> +++ b/package/matchbox/matchbox.mk
> @@ -11,22 +11,11 @@ MATCHBOX_LICENSE = GPLv2+
> MATCHBOX_LICENSE_FILES = COPYING
>
> MATCHBOX_DEPENDENCIES = matchbox-lib
> -MATCHBOX_CONF_OPTS = --enable-expat
> +MATCHBOX_CONF_OPTS = --enable-expat --disable-composite
>
> # Workaround bug in configure script
> MATCHBOX_CONF_ENV = expat=yes
>
> -ifeq ($(BR2_PACKAGE_XLIB_LIBXCOMPOSITE)$(BR2_PACKAGE_XLIB_LIBXFIXES)$(BR2_PACKAGE_XLIB_LIBXDAMAGE)$(BR2_PACKAGE_XLIB_LIBXRENDER),yyyy)
> -MATCHBOX_CONF_OPTS += --enable-composite
> -MATCHBOX_DEPENDENCIES += \
> - xlib_libXcomposite \
> - xlib_libXfixes \
> - xlib_libXdamage \
> - xlib_libXrender
> -else
> -MATCHBOX_CONF_OPTS += --disable-composite
> -endif
> -
> ifeq ($(BR2_PACKAGE_XLIB_LIBXFT),y)
> MATCHBOX_DEPENDENCIES += xlib_libXft
> endif
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list