diff mbox series

[wrynose] weston: fix VNC backend build with aml 1.0

Message ID 20261008101805.2770110-1-frieder@fris.de
State New
Headers show
Series [wrynose] weston: fix VNC backend build with aml 1.0 | expand

Commit Message

Frieder Schrempf Oct. 8, 2026, 10:18 a.m. UTC
From: Frieder Schrempf <frieder.schrempf@kontron.de>

meta-oe's aml 1.0 installs aml1.pc, but weston 15 looks for aml 0.3.x, so
enabling the vnc PACKAGECONFIG fails at configure time. Backport the aml
part of the upstream fix from weston 16 and add aml to the vnc
dependencies, as weston uses it directly.

Tested by building weston with PACKAGECONFIG vnc for ARMv7.

AI-Generated: Uses Claude Code (Claude Opus 5.5)
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 .../weston/0001-backend-vnc-Use-aml-v1.patch  | 23 +++++++++++++++++++
 .../recipes-graphics/wayland/weston_15.0.0.bb |  3 ++-
 2 files changed, 25 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch

Comments

Yoann Congal Oct. 8, 2026, 10:27 a.m. UTC | #1
On Thu Oct 8, 2026 at 12:18 PM CEST, Frieder Schrempf via lists.openembedded.org wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> meta-oe's aml 1.0 installs aml1.pc, but weston 15 looks for aml 0.3.x, so
> enabling the vnc PACKAGECONFIG fails at configure time. Backport the aml
> part of the upstream fix from weston 16 and add aml to the vnc
> dependencies, as weston uses it directly.
>
> Tested by building weston with PACKAGECONFIG vnc for ARMv7.
>
> AI-Generated: Uses Claude Code (Claude Opus 5.5)
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
>  .../weston/0001-backend-vnc-Use-aml-v1.patch  | 23 +++++++++++++++++++
>  .../recipes-graphics/wayland/weston_15.0.0.bb |  3 ++-
>  2 files changed, 25 insertions(+), 1 deletion(-)
>  create mode 100644 meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch
>
> diff --git a/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch b/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch
> new file mode 100644
> index 0000000000..b767c9ac07
> --- /dev/null
> +++ b/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch
> @@ -0,0 +1,23 @@
> +From: Frieder Schrempf <frieder.schrempf@kontron.de>
> +Subject: [PATCH] backend-vnc: Use aml v1
> +
> +aml 1.0 installs aml1.pc and neatvnc already uses it. Only the aml part
> +of the upstream commit is taken, as the rest needs Neat VNC 1.0.
> +
> +Upstream-Status: Backport [partial, https://gitlab.freedesktop.org/wayland/weston/-/commit/8a1c91e7]
> +---
Hello,

This patch lacks your signed-off-by.
Can you send a v2 with it?

The rest of the patch looks good at first glance.

Thanks!
diff mbox series

Patch

diff --git a/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch b/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch
new file mode 100644
index 0000000000..b767c9ac07
--- /dev/null
+++ b/meta/recipes-graphics/wayland/weston/0001-backend-vnc-Use-aml-v1.patch
@@ -0,0 +1,23 @@ 
+From: Frieder Schrempf <frieder.schrempf@kontron.de>
+Subject: [PATCH] backend-vnc: Use aml v1
+
+aml 1.0 installs aml1.pc and neatvnc already uses it. Only the aml part
+of the upstream commit is taken, as the rest needs Neat VNC 1.0.
+
+Upstream-Status: Backport [partial, https://gitlab.freedesktop.org/wayland/weston/-/commit/8a1c91e7]
+---
+ libweston/backend-vnc/meson.build | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/libweston/backend-vnc/meson.build b/libweston/backend-vnc/meson.build
+--- a/libweston/backend-vnc/meson.build
++++ b/libweston/backend-vnc/meson.build
+@@ -8,7 +8,7 @@ if not dep_neatvnc.found()
+ 	error('VNC backend requires neatvnc which was not found. Or, you can use \'-Dbackend-vnc=false\'.')
+ endif
+ 
+-dep_aml = dependency('aml', version: ['>= 0.3.0', '< 0.4.0'], required: false, fallback: ['aml', 'aml_dep'])
++dep_aml = dependency('aml1', version: ['>= 1.0.0', '< 2.0.0'], required: false, fallback: ['aml', 'aml_dep'])
+ if not dep_aml.found()
+ 	error('VNC backend requires libaml which was not found. Or, you can use \'-Dbackend-vnc=false\'.')
+ endif
diff --git a/meta/recipes-graphics/wayland/weston_15.0.0.bb b/meta/recipes-graphics/wayland/weston_15.0.0.bb
index da347659f1..3f9fff51fd 100644
--- a/meta/recipes-graphics/wayland/weston_15.0.0.bb
+++ b/meta/recipes-graphics/wayland/weston_15.0.0.bb
@@ -9,6 +9,7 @@  LIC_FILES_CHKSUM = "file://COPYING;md5=d79ee9e66bb0f95d3386a7acae780b70 \
 SRC_URI = "https://gitlab.freedesktop.org/wayland/weston/-/releases/${PV}/downloads/${BPN}-${PV}.tar.xz \
            file://0001-vulkan-renderer-guard-surface-output-creation-with-b.patch \
            file://0001-gl-shaders-Remove-asserts-relying-on-shader-compiler.patch \
+           file://0001-backend-vnc-Use-aml-v1.patch \
            file://weston.png \
            file://weston.desktop \
            file://xwayland.weston-start \
@@ -98,7 +99,7 @@  PACKAGECONFIG[image-jpeg] = "-Dimage-jpeg=true,-Dimage-jpeg=false, jpeg"
 # screencasting via PipeWire
 PACKAGECONFIG[pipewire] = "-Dbackend-pipewire=true,-Dbackend-pipewire=false,pipewire,pipewire"
 # VNC remote screensharing
-PACKAGECONFIG[vnc] = "-Dbackend-vnc=true,-Dbackend-vnc=false,neatvnc libpam"
+PACKAGECONFIG[vnc] = "-Dbackend-vnc=true,-Dbackend-vnc=false,neatvnc aml libpam"
 # Perfetto performance analysis support
 PACKAGECONFIG[perfetto] = "-Dperfetto=true,-Dperfetto=false,libperfetto"