Source: weston
Version: 10.0.1-1
Severity: wishlist
Tags: patch
X-Debbugs-Cc: [email protected]

Dear Maintainers, hi.

First of all, thanks for maintaining weston in Debian.

Weston 10 is using the default build options set by upstream plus
screenshare and systemd features enabled via a debian/rules override of
dh_auto_configure. This basically covers the major cases for PCs and
servers, but for embedded systems and people not wanting to use systemd
(and logind), this is a problem.

Weston has support for using libseat for seat management/launching
weston via the launcher-libseat option (it's turned off by default in
meson_options.txt), but weston 11 libseat launcher became the default [1]
and recently (weston 12) the libseat launcher has become the one and
only official launcher supported by upstream weston [2].

Taking this into consideration and the outcome of Init systems and
systemd GR [3], I'd like to check with you the possibility of enabling the
support for libseat launcher in stable via a proposed-updates upload to
bookworm.

I did rebuild bookworm's version with the option enabled and tested it a
bit in my notebook. I've also used abi-compliance-checker to check if
this could have caused any ABI change in libweston and it didn't. I'll
provide the debdiff output for you to check.

Also, should you answer positively, I can do a more exhaustive testing
and check if we don't introduce any errors when running with elogind and
systemd-logind.

Cheers,
Charles

[1] 
https://www.collabora.com/news-and-blog/news-and-events/weston-11-whats-new-whats-next.html
[2] 
https://www.collabora.com/news-and-blog/news-and-events/weston-12-highlights-and-changes.html
[3] https://www.debian.org/vote/2019/vote_002

P.S. I don't think this system info is going to help since I've built in
bookworm chroot.

-- System Information:
Debian Release: trixie/sid
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 6.6.15-amd64 (SMP w/8 CPU threads; PREEMPT)
Locale: LANG=pt_BR.UTF-8, LC_CTYPE=pt_BR.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US:en
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
diff -u weston-10.0.1/debian/changelog weston-10.0.1/debian/changelog
--- weston-10.0.1/debian/changelog
+++ weston-10.0.1/debian/changelog
@@ -1,3 +1,11 @@
+weston (10.0.1-1+deb12u1) bookworm; urgency=medium
+
+  * Team upload.
+  * debian/control: add libseat-dev to Build-Depends.
+  * debian/rules: enable libseat support. (Closes: #<bug_number>)
+
+ -- Carlos Henrique Lima Melara <[email protected]>  Tue, 12 Mar 2024 
12:54:10 -0300
+
 weston (10.0.1-1) unstable; urgency=medium
 
   * New upstream release.
diff -u weston-10.0.1/debian/control weston-10.0.1/debian/control
--- weston-10.0.1/debian/control
+++ weston-10.0.1/debian/control
@@ -26,6 +26,7 @@
                libpixman-1-dev (>= 0.25.2),
                libpipewire-0.3-dev,
                libpng-dev,
+               libseat-dev,
                libsystemd-dev,
                libudev-dev (>= 136),
                libva-dev,
diff -u weston-10.0.1/debian/rules weston-10.0.1/debian/rules
--- weston-10.0.1/debian/rules
+++ weston-10.0.1/debian/rules
@@ -2,7 +2,8 @@
 
 override_dh_auto_configure:
        dh_auto_configure -- -Dscreenshare=true \
-               -Dsystemd=true
+               -Dsystemd=true \
+               -Dlauncher-libseat=true
 
 override_dh_auto_test:
        mkdir -p $(CURDIR)/debian/tmp/tmp/xdgruntimedir

Reply via email to