[PATCH] libinput: Add runtime dependencies for 'libinput measure' command

[PATCH] libinput: Add runtime dependencies for 'libinput measure' command

[Bhargav Das]

From: Bhargav Das <bhargav.das@siemens.com>

When using the 'measure' subcommand such as,

libinput measure touchpad-size 800x480 /dev/input/event0

The following runtime dependencies are required.
* python3i,
* python3-libevdev,
* python3-pyudev

Signed-off-by: Bhargav Das <bhargav.das@siemens.com>
---
 meta/recipes-graphics/wayland/libinput_1.26.1.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-graphics/wayland/libinput_1.26.1.bb b/meta/recipes-graphics/wayland/libinput_1.26.1.bb
index 487270505a..74fbbdb000 100644
--- a/meta/recipes-graphics/wayland/libinput_1.26.1.bb
+++ b/meta/recipes-graphics/wayland/libinput_1.26.1.bb
@@ -46,4 +46,6 @@ RPROVIDES:${PN} = "libinput"
 RREPLACES:${PN} = "libinput"
 RCONFLICTS:${PN} = "libinput"
 
+RDEPENDS:${PN} = "python3 python3-libevdev python3-pyudev"
+
 FILES:${PN}-ptest += "${libexecdir}/libinput/libinput-test-suite"
-- 
2.34.1

Re: [OE-core] [PATCH] libinput: Add runtime dependencies for 'libinput measure' command

[Alexander Kanavin]

On Wed, 23 Oct 2024 at 16:31, Bhargav Das via lists.openembedded.org
<bhargav_das=mentor.com@lists.openembedded.org> wrote:
> +RDEPENDS:${PN} = "python3 python3-libevdev python3-pyudev"

This will add python dependency to a very core package, and in
embedded context we cannot do that.

Rather, the real issue is that libinput component is incorrectly
packaged: library and items in /usr/libexec go to libinput, executable
in /usr/bin using those items goes to libinput-bin. Instead, libexec
items should be moved to libinput-bin, and then that package should
gain the python dependency.

Alex

Re: [PATCH] libinput: Add runtime dependencies for 'libinput measure' command

[Bhargav Das]

[-- Attachment #1: Type: text/plain, Size: 175 bytes --]

Thanks for the explanation and the drawbacks of adding this type of dependencies. I haven't checked libinput-bin at all. I will look at it though but it might take sometime.

[-- Attachment #2: Type: text/html, Size: 186 bytes --]