[Buildroot] [PATCH v2 1/1] package/weston: replace autotools with meson
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sat Aug 3 22:36:05 UTC 2019
Hello James,
On Mon, 24 Jun 2019 12:11:45 -0600
James Hilliard <james.hilliard1 at gmail.com> wrote:
> config BR2_PACKAGE_WESTON_DEMO_CLIENTS
> bool "demo clients"
> + depends on BR2_PACKAGE_MESA3D_OPENGL_EGL
Why is this option added as a dependency. I understand about the other
depends on and the select on pango, but I don't understand this
BR2_PACKAGE_MESA3D_OPENGL_EGL dependency.
> + depends on BR2_USE_WCHAR # pango
> + depends on BR2_TOOLCHAIN_HAS_THREADS # pango
> + depends on BR2_USE_MMU # pango
> + depends on BR2_INSTALL_LIBSTDCPP # pango
> + depends on BR2_TOOLCHAIN_HAS_SYNC_4 # pango
> + select BR2_PACKAGE_PANGO
> help
> This enables the installation of Weston's demo clients.
> +# Uses VIDIOC_EXPBUF, only available from 3.8+
> +ifeq ($(BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_8),)
If you want to say "build this when headers >= 3.8", then it should be:
ifeq ($(BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_8),y)
> +WESTON_CONF_OPTS += -Dsimple-clients=dmabuf-v4l
> +else
> +WESTON_CONF_OPTS += -Dsimple-clients=
> endif
>
> ifeq ($(BR2_PACKAGE_WESTON_FBDEV),y)
> WESTON_CONF_OPTS += \
> - --enable-fbdev-compositor \
> - WESTON_NATIVE_BACKEND=fbdev-backend.so
> + -Dbackend-fbdev=true \
> + -Dbackend-default=fbdev
I'm still not happy with the -Dbackend-default handling. Indeed, if you
enable fbdev, and drm and x11, and xwayland, you are going to pass 4
-Dbackend-default options.
Is this backend-default option mandatory ? Perhaps we can leave it
undefined, and simply have the user select the backend at runtime ? I
suppose if there is only backend, weston is smart enough at runtime to
use that one ?
Could you test with this -Dbackend-default option, making a runtime
test ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list