[Debian-on-mobile-maintainers] [Git][DebianOnMobile-team/phosh][debian/latest] 2 commits: dbus: Add Backlight property to DisplayConfig

[Guido Günther (@agx)]

Guido Günther pushed to branch debian/latest at Debian On Mobile / phosh


Commits:
b729056a by Guido Günther at 2024-09-14T16:11:32+02:00
dbus: Add Backlight property to DisplayConfig

This prevents gsd-power 47~rc1 from crashing

- - - - -
0a772f29 by Guido Günther at 2024-09-14T16:12:33+02:00
Document changes and release 0.41.1-2

- - - - -


3 changed files:

- debian/changelog
- + debian/patches/dbus-Add-Backlight-property-to-DisplayConfig.patch
- debian/patches/series


Changes:

=====================================
debian/changelog
=====================================
@@ -1,3 +1,10 @@
+phosh (0.41.1-2) unstable; urgency=medium
+
+  * dbus: Add Backlight property to DisplayConfig.
+    This prevents gsd-power 47~rc1 from crashing
+
+ -- Guido Günther <agx at sigxcpu.org>  Sat, 14 Sep 2024 16:12:08 +0200
+
 phosh (0.41.1-1) unstable; urgency=medium
 
   * New upstream bugfix release


=====================================
debian/patches/dbus-Add-Backlight-property-to-DisplayConfig.patch
=====================================
@@ -0,0 +1,44 @@
+From: =?utf-8?q?Guido_G=C3=BCnther?= <agx at sigxcpu.org>
+Date: Fri, 13 Sep 2024 13:49:03 +0200
+Subject: dbus: Add Backlight property to DisplayConfig
+MIME-Version: 1.0
+Content-Type: text/plain; charset="utf-8"
+Content-Transfer-Encoding: 8bit
+
+This ensures GNOME 47's gsd-power can read it and doesn't crash.
+
+Having it empty is fine for the moment.
+
+Signed-off-by: Guido Günther <agx at sigxcpu.org>
+Part-of: <https://gitlab.gnome.org/World/Phosh/phosh/-/merge_requests/1526>
+---
+ src/dbus/org.gnome.Mutter.DisplayConfig.xml | 16 ++++++++++++++++
+ 1 file changed, 16 insertions(+)
+
+diff --git a/src/dbus/org.gnome.Mutter.DisplayConfig.xml b/src/dbus/org.gnome.Mutter.DisplayConfig.xml
+index 0501908..fe0d106 100644
+--- a/src/dbus/org.gnome.Mutter.DisplayConfig.xml
++++ b/src/dbus/org.gnome.Mutter.DisplayConfig.xml
+@@ -216,6 +216,22 @@
+       <arg name="new_value" direction="out" type="i" />
+     </method>
+ 
++    <!--
++	Backlight:
++
++	A set of backlights. Each backlight is a dictionary with the following
++	entries. If an entry is only a connector, it means there is no way to
++	control it via this D-Bus interface.
++
++	  * 'connector' (s) - An associated monitor connector
++	  * 'active' (s) - True if the monitor is active
++	  * 'min_step' (i) - Minimum step amount (optional)
++	  * 'value' (i) - Current value (optional)
++
++	The initial 'u' is a serial number used when setting the backlight.
++    -->
++    <property name="Backlight" type="(uaa{sv})" access="read" />
++
+     <!--
+         GetCrtcGamma:
+ 	@serial: configuration serial


=====================================
debian/patches/series
=====================================
@@ -1 +1,2 @@
 tests-Skip-symbol-check-on-some-architectures.patch
+dbus-Add-Backlight-property-to-DisplayConfig.patch



View it on GitLab: https://salsa.debian.org/DebianOnMobile-team/phosh/-/compare/6cf7417fb2d0de4c962d1235bb7f3aa35b743336...0a772f2936d65d3269f9ee4e0af582717883ffc1

-- 
View it on GitLab: https://salsa.debian.org/DebianOnMobile-team/phosh/-/compare/6cf7417fb2d0de4c962d1235bb7f3aa35b743336...0a772f2936d65d3269f9ee4e0af582717883ffc1
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-on-mobile-maintainers/attachments/20240915/20fb9f94/attachment-0001.htm>