diff mbox series

libinput: Add runtime dependencies for 'libinput measure' command

Message ID 20241023142730.477569-1-bhargav_das@mentor.com
State New
Headers show
Series libinput: Add runtime dependencies for 'libinput measure' command | expand

Commit Message

Bhargav Das Oct. 23, 2024, 2:27 p.m. UTC 
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(+)

Comments

Alexander Kanavin Oct. 23, 2024, 2:50 p.m. UTC | #1 
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
Bhargav Das Oct. 23, 2024, 4:01 p.m. UTC | #2 
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.
diff mbox series

Patch

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"