* [Buildroot] [PATCH 2/2] package/weston: introduce _SIMPLE_CLIENTS package option
2023-08-03 20:15 [Buildroot] [PATCH 1/2] package/weston: fix the configuration of simple-clients Julien Olivain
@ 2023-08-03 20:15 ` Julien Olivain
2023-08-12 18:35 ` Yann E. MORIN
2023-08-12 18:33 ` [Buildroot] [PATCH 1/2] package/weston: fix the configuration of simple-clients Yann E. MORIN
1 sibling, 1 reply; 4+ messages in thread
From: Julien Olivain @ 2023-08-03 20:15 UTC (permalink / raw)
To: buildroot; +Cc: Julien Olivain, Yann E . MORIN, Thomas Petazzoni
commit "package/weston: fix the configuration of simple-clients"
fixed the configuration of Weston simple-clients by always enabling all
supported applications.
Since it is not desirable to always have all those applications
installed on target, this commit introduces a new package Kconfig
option to control this.
Since there is already a BR2_PACKAGE_WESTON_DEMO_CLIENTS option present,
this commit also update the Kconfig "help" entries, to clarify the
difference between those "demo clients" and "simple clients". Those
clients groups are directly coming from the upstream package (and are
directly mapped to Meson configure options).
Signed-off-by: Julien Olivain <ju.o@free.fr>
---
package/weston/Config.in | 24 ++++++++++++++++++++++++
package/weston/weston.mk | 4 ++++
2 files changed, 28 insertions(+)
diff --git a/package/weston/Config.in b/package/weston/Config.in
index 1a7c27a715..db6a78ee8a 100644
--- a/package/weston/Config.in
+++ b/package/weston/Config.in
@@ -174,6 +174,18 @@ config BR2_PACKAGE_WESTON_SCREENSHARE
bool "screenshare"
default y
+config BR2_PACKAGE_WESTON_SIMPLE_CLIENTS
+ bool "simple clients"
+ help
+ Simple clients includes application such as:
+ damage, dmabuf-feedback, dmabuf-egl, dmabuf-v4l, egl
+
+ In the target filesystem, simple client program binaries are
+ prefixed with "weston-simple-".
+
+ Note: Weston has two sets of clients. See also the weston
+ "demo clients" option.
+
config BR2_PACKAGE_WESTON_DEMO_CLIENTS
bool "demo clients"
depends on BR2_USE_MMU # pango
@@ -188,6 +200,18 @@ config BR2_PACKAGE_WESTON_DEMO_CLIENTS
help
This enables the installation of Weston's demo clients.
+ Demo clients includes application such as:
+ clickdot, confine, content_protection, dnd, editor,
+ eventdemo, flower, fullscreen, image, multi-resource,
+ presentation-shm, resizor, scaler, smoke, stacking,
+ subsurfaces, tablet, transformed...
+
+ In the target filesystem, demo client program binaries are
+ prefixed with "weston-".
+
+ Note: Weston has two sets of clients. See also the weston
+ "simple clients" option.
+
comment "demo clients needs an OpenGL ES provider, an OpenEGL-capable Wayland backend and a toolchain w/ wchar, threads, C++, gcc >= 4.9"
depends on BR2_USE_MMU
depends on BR2_TOOLCHAIN_HAS_SYNC_4
diff --git a/package/weston/weston.mk b/package/weston/weston.mk
index 5f8d915d5a..0974159a9f 100644
--- a/package/weston/weston.mk
+++ b/package/weston/weston.mk
@@ -22,6 +22,7 @@ WESTON_CONF_OPTS = \
-Dlauncher-libseat=true \
-Dtools=calibrator,debug,info,terminal,touch-calibrator
+ifeq ($(BR2_PACKAGE_WESTON_SIMPLE_CLIENTS),y)
WESTON_SIMPLE_CLIENTS = \
damage \
dmabuf-egl \
@@ -37,6 +38,9 @@ WESTON_SIMPLE_CLIENTS += dmabuf-v4l
endif
WESTON_CONF_OPTS += -Dsimple-clients=$(subst $(space),$(comma),$(strip $(WESTON_SIMPLE_CLIENTS)))
+else
+WESTON_CONF_OPTS += -Dsimple-clients=
+endif
ifeq ($(BR2_PACKAGE_JPEG),y)
WESTON_CONF_OPTS += -Dimage-jpeg=true
--
2.41.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATCH 1/2] package/weston: fix the configuration of simple-clients
2023-08-03 20:15 [Buildroot] [PATCH 1/2] package/weston: fix the configuration of simple-clients Julien Olivain
2023-08-03 20:15 ` [Buildroot] [PATCH 2/2] package/weston: introduce _SIMPLE_CLIENTS package option Julien Olivain
@ 2023-08-12 18:33 ` Yann E. MORIN
1 sibling, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2023-08-12 18:33 UTC (permalink / raw)
To: Julien Olivain; +Cc: Thomas Petazzoni, buildroot
Julien, All,
On 2023-08-03 22:15 +0200, Julien Olivain spake thusly:
> commit 7329a0db442026c6b856e76ea964107fa3c730f3
> "weston: disable v4l simple dmabuf for headers < 3.8"
> disabled dmabuf-v4l-client when kernel headers are older than v3.8.
> This commit was made at the time the weston package was at version
> 1.10.0, using the autotools package infra.
>
> commit e672eb5c39fa626f8e924269cec2056be9e14ded
> "package/weston: bump to version 8.0.0"
> replaced the package infra to meson (since upstream deprecated
> autotools). This commit changed a disable of dmabuf-v4l-client in
> autotools, by enabling it only if supported. The end result is that
> the current meson package recipe disable ALL simple clients, or just
> enable dmabuf-v4l-client when supported. In all cases, all other
> weston simple clients are disabled. This behavior is not convenient
> since some of those simple clients are useful to test the correct
> operation of Weston and the rest of the graphic stack.
>
> This commit fixes this issue by enabling all supported simple clients.
>
> Signed-off-by: Julien Olivain <ju.o@free.fr>
Applied to next, thanks.
Regards,
Yann E. MORIN.
> ---
> package/weston/weston.mk | 17 +++++++++++++----
> 1 file changed, 13 insertions(+), 4 deletions(-)
>
> diff --git a/package/weston/weston.mk b/package/weston/weston.mk
> index 437c6f2e38..5f8d915d5a 100644
> --- a/package/weston/weston.mk
> +++ b/package/weston/weston.mk
> @@ -22,13 +22,22 @@ WESTON_CONF_OPTS = \
> -Dlauncher-libseat=true \
> -Dtools=calibrator,debug,info,terminal,touch-calibrator
>
> -# Uses VIDIOC_EXPBUF, only available from 3.8+
> +WESTON_SIMPLE_CLIENTS = \
> + damage \
> + dmabuf-egl \
> + dmabuf-feedback \
> + egl \
> + im \
> + shm \
> + touch
> +
> ifeq ($(BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_8),y)
> -WESTON_CONF_OPTS += -Dsimple-clients=dmabuf-v4l
> -else
> -WESTON_CONF_OPTS += -Dsimple-clients=
> +# dmabuf-v4l uses VIDIOC_EXPBUF, only available from 3.8+
> +WESTON_SIMPLE_CLIENTS += dmabuf-v4l
> endif
>
> +WESTON_CONF_OPTS += -Dsimple-clients=$(subst $(space),$(comma),$(strip $(WESTON_SIMPLE_CLIENTS)))
> +
> ifeq ($(BR2_PACKAGE_JPEG),y)
> WESTON_CONF_OPTS += -Dimage-jpeg=true
> WESTON_DEPENDENCIES += jpeg
> --
> 2.41.0
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 4+ messages in thread