[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1108213: marked as done (unblock: lomiri-system-settings/1.3.1-6)



Your message dated Wed, 25 Jun 2025 10:42:04 +0000
with message-id <E1uUNZs-00DnBA-0f@respighi.debian.org>
and subject line unblock lomiri-system-settings
has caused the Debian Bug report #1108213,
regarding unblock: lomiri-system-settings/1.3.1-6
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1108213: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1108213
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
X-Debbugs-Cc: lomiri-system-settings@packages.debian.org
Control: affects -1 + src:lomiri-system-settings
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package lomiri-system-settings

The past days I have been looking into issue for people you rely on
switching between keyboard layout for the daily computer work because of
having to switch between language contexts (such as using a computer for
English and Russian or Ukrainian communication). For those people it is
essential of layout switches (for hardware keyboard as well as for OSK)
easily accessible. (OSK - On-Screen Keyboard).

[ Reason ]

+  * debian/rules:
+    + Set correct lomiri-keyboard plugin-path at build-time.

-> This build-time fix lets lomiri-system-settings find the install
lomiri-keyboard-<lang> plugin files. Without this, the settings page for
OSK layout selection is empty.

+ -- Mike Gabriel <sunweaver@debian.org>  Mon, 23 Jun 2025 13:23:09 +0200
+
+lomiri-system-settings (1.3.1-5) unstable; urgency=medium
+
+  * debian/patches:
+    + Replace 0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-
+      missing-D-Bus-types.patch by 0001_plugins-language-hardwarekeyboard-
+      plugin.cpp-Registe.patch (reviewed upstream patch).

-> After upstream review, patch 0001 has now been replaced with the
official upstream patch.

+    + Add 0002_Add-shortcut-to-OSK-layout-settings.patch. Make OSK layout
+      settings available via URL dispatcher call.

-> This update of lomiri-system-settings will come together with an update
of ayatana-indicator-keyboard (still to be finalized). For that change in
ayatana-indicator-keyboard, we have to make sure that OSK layout settings
can be evoked by Lomiri's URL dispatcher service.

[ Impact ]
Broken OSK-support in Debian 13's version of Lomiri.

[ Tests ]
Manual tests on various devices (tablet, tablet with keyboard attached,
notebook, VM, etc.).

[ Risks ]
Minimal.

[ Checklist ]
  [x] all changes are documented in the d/changelog
  [x] I reviewed all changes and I approve them
  [x] attach debdiff against the package in testing

[ Other info ]
Relevant for Lomiri in Debian.

unblock lomiri-system-settings/1.3.1-6
diff -Nru lomiri-system-settings-1.3.1/debian/changelog lomiri-system-settings-1.3.1/debian/changelog
--- lomiri-system-settings-1.3.1/debian/changelog	2025-06-14 07:54:49.000000000 +0200
+++ lomiri-system-settings-1.3.1/debian/changelog	2025-06-23 13:23:09.000000000 +0200
@@ -1,3 +1,21 @@
+lomiri-system-settings (1.3.1-6) unstable; urgency=medium
+
+  * debian/rules:
+    + Set correct lomiri-keyboard plugin-path at build-time.
+
+ -- Mike Gabriel <sunweaver@debian.org>  Mon, 23 Jun 2025 13:23:09 +0200
+
+lomiri-system-settings (1.3.1-5) unstable; urgency=medium
+
+  * debian/patches:
+    + Replace 0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-
+      missing-D-Bus-types.patch by 0001_plugins-language-hardwarekeyboard-
+      plugin.cpp-Registe.patch (reviewed upstream patch).
+    + Add 0002_Add-shortcut-to-OSK-layout-settings.patch. Make OSK layout
+      settings available via URL dispatcher call.
+
+ -- Mike Gabriel <sunweaver@debian.org>  Sun, 22 Jun 2025 22:16:23 +0200
+
 lomiri-system-settings (1.3.1-4) unstable; urgency=medium
 
   * debian/patches:
diff -Nru lomiri-system-settings-1.3.1/debian/patches/0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-missing-D-Bus-types.patch lomiri-system-settings-1.3.1/debian/patches/0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-missing-D-Bus-types.patch
--- lomiri-system-settings-1.3.1/debian/patches/0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-missing-D-Bus-types.patch	2025-06-14 07:29:15.000000000 +0200
+++ lomiri-system-settings-1.3.1/debian/patches/0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-missing-D-Bus-types.patch	1970-01-01 01:00:00.000000000 +0100
@@ -1,47 +0,0 @@
-From bbb79805a1bf76402774164eece1e69396329c5e Mon Sep 17 00:00:00 2001
-From: Robert Tari <robert@tari.in>
-Date: Fri, 13 Jun 2025 14:56:47 +0200
-Subject: [PATCH] lib/LomiriSystemSettingsPrivate/accountsservice.cpp: Register
- missing D-Bus types
-
-fixes https://salsa.debian.org/ubports-team/lomiri-system-settings/-/issues/19
----
- lib/LomiriSystemSettingsPrivate/accountsservice.cpp | 9 ++++++++-
- 1 file changed, 8 insertions(+), 1 deletion(-)
-
-diff --git a/lib/LomiriSystemSettingsPrivate/accountsservice.cpp b/lib/LomiriSystemSettingsPrivate/accountsservice.cpp
-index 6450a4790..3d5383e1a 100644
---- a/lib/LomiriSystemSettingsPrivate/accountsservice.cpp
-+++ b/lib/LomiriSystemSettingsPrivate/accountsservice.cpp
-@@ -22,7 +22,7 @@
- 
- #include <QDBusReply>
- #include <QDebug>
--
-+#include <QDBusMetaType>
- #include <unistd.h>
- #include <sys/types.h>
- 
-@@ -30,6 +30,11 @@
- #define AS_PATH "/org/freedesktop/Accounts"
- #define AS_IFACE "org.freedesktop.Accounts"
- 
-+using StringMap = QMap<QString,QString>;
-+using StringMapList = QList<StringMap>;
-+Q_DECLARE_METATYPE(StringMap)
-+Q_DECLARE_METATYPE(StringMapList)
-+
- namespace LomiriSystemSettings {
- 
- AccountsService::AccountsService(QObject *parent)
-@@ -43,6 +48,8 @@ AccountsService::AccountsService(QObject *parent)
-                              AS_IFACE,
-                              m_systemBusConnection)
- {
-+    qDBusRegisterMetaType<StringMap>();
-+    qDBusRegisterMetaType<StringMapList>();
-     connect (&m_serviceWatcher,
-              SIGNAL (serviceOwnerChanged (QString, QString, QString)),
-              this,
--- 
-GitLab
diff -Nru lomiri-system-settings-1.3.1/debian/patches/0001_plugins-language-hardwarekeyboard-plugin.cpp-Registe.patch lomiri-system-settings-1.3.1/debian/patches/0001_plugins-language-hardwarekeyboard-plugin.cpp-Registe.patch
--- lomiri-system-settings-1.3.1/debian/patches/0001_plugins-language-hardwarekeyboard-plugin.cpp-Registe.patch	1970-01-01 01:00:00.000000000 +0100
+++ lomiri-system-settings-1.3.1/debian/patches/0001_plugins-language-hardwarekeyboard-plugin.cpp-Registe.patch	2025-06-15 20:52:42.000000000 +0200
@@ -0,0 +1,38 @@
+From 458a69fbe87fae55d842d586d2993fc1c29d7485 Mon Sep 17 00:00:00 2001
+From: Robert Tari <robert@tari.in>
+Date: Fri, 13 Jun 2025 14:56:47 +0200
+Subject: [PATCH] plugins/language/hardwarekeyboard-plugin.cpp: Register
+ missing D-Bus types
+
+fixes https://salsa.debian.org/ubports-team/lomiri-system-settings/-/issues/19
+
+Signed-off-by: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
+---
+ plugins/language/hardwarekeyboard-plugin.cpp | 5 ++++-
+ 1 file changed, 4 insertions(+), 1 deletion(-)
+
+diff --git a/plugins/language/hardwarekeyboard-plugin.cpp b/plugins/language/hardwarekeyboard-plugin.cpp
+index a74ed409..6317922c 100644
+--- a/plugins/language/hardwarekeyboard-plugin.cpp
++++ b/plugins/language/hardwarekeyboard-plugin.cpp
+@@ -31,13 +31,16 @@
+ #define SOURCES_CONFIG_SCHEMA_ID "org.gnome.desktop.input-sources"
+ #define SOURCES_KEY "sources"
+ 
+-typedef QList<QMap<QString, QString>> StringMapList;
++using StringMap = QMap<QString,QString>;
++using StringMapList = QList<StringMap>;
++Q_DECLARE_METATYPE(StringMap)
+ Q_DECLARE_METATYPE(StringMapList)
+ 
+ HardwareKeyboardPlugin::HardwareKeyboardPlugin(QObject *parent) :
+     QObject(parent),
+     m_sourcesSettings(g_settings_new(SOURCES_CONFIG_SCHEMA_ID))
+ {
++    qDBusRegisterMetaType<StringMap>();
+     qDBusRegisterMetaType<StringMapList>();
+     m_xkbInfo = gnome_xkb_info_new();
+ 
+-- 
+2.47.2
+
diff -Nru lomiri-system-settings-1.3.1/debian/patches/0002_Add-shortcut-to-OSK-layout-settings.patch lomiri-system-settings-1.3.1/debian/patches/0002_Add-shortcut-to-OSK-layout-settings.patch
--- lomiri-system-settings-1.3.1/debian/patches/0002_Add-shortcut-to-OSK-layout-settings.patch	1970-01-01 01:00:00.000000000 +0100
+++ lomiri-system-settings-1.3.1/debian/patches/0002_Add-shortcut-to-OSK-layout-settings.patch	2025-06-22 22:14:53.000000000 +0200
@@ -0,0 +1,42 @@
+From bcdf4a63bbaccad40ce795159344a1457a63b323 Mon Sep 17 00:00:00 2001
+From: Robert Tari <robert@tari.in>
+Date: Sun, 15 Jun 2025 14:28:26 +0200
+Subject: [PATCH] Add a shortcut to OSK layout settings
+
+---
+ plugins/language/PageComponent.qml | 7 +++++++
+ src/url-map.ini                    | 1 +
+ 2 files changed, 8 insertions(+)
+
+diff --git a/plugins/language/PageComponent.qml b/plugins/language/PageComponent.qml
+index 02c0369e2..3c37bf6bc 100644
+--- a/plugins/language/PageComponent.qml
++++ b/plugins/language/PageComponent.qml
+@@ -54,6 +54,13 @@ ItemPage {
+                     currentLayoutsDraggable: true
+                 });
+                 break;
++            case 'sw-keyboard-layouts':
++                pageStack.addPageToNextColumn(
++                    root, Qt.resolvedUrl('KeyboardLayouts.qml'), {
++                    plugin: oskPlugin,
++                    currentLayoutsDraggable: true
++                });
++                break;
+             }
+         }
+     }
+diff --git a/src/url-map.ini b/src/url-map.ini
+index 1bcf39a0f..567e85b11 100644
+--- a/src/url-map.ini
++++ b/src/url-map.ini
+@@ -10,5 +10,6 @@
+ location=settings:///system/security-privacy?subpage=location
+ permissions=settings:///system/security-privacy?subpage=permissions
+ hw-keyboard-layouts=settings:///system/language?subpage=hw-keyboard-layouts
++sw-keyboard-layouts=settings:///system/language?subpage=sw-keyboard-layouts
+ add-printer=settings:///system/printing?subpage=add-printer
+ view-printer=settings:///system/printing?subpage=view-printer
+-- 
+GitLab
+
diff -Nru lomiri-system-settings-1.3.1/debian/patches/series lomiri-system-settings-1.3.1/debian/patches/series
--- lomiri-system-settings-1.3.1/debian/patches/series	2025-06-14 07:29:15.000000000 +0200
+++ lomiri-system-settings-1.3.1/debian/patches/series	2025-06-22 22:15:17.000000000 +0200
@@ -2,4 +2,5 @@
 2011_build-without-trust-store.patch
 2013_show-hotspot-on-desktop.patch
 2014_removing-unnecessary-mobile-only-functionality.patch
-0001_lib-LomiriSystemSettingsPrivate-accountsservice.cpp-Register-missing-D-Bus-types.patch
+0001_plugins-language-hardwarekeyboard-plugin.cpp-Registe.patch
+0002_Add-shortcut-to-OSK-layout-settings.patch
diff -Nru lomiri-system-settings-1.3.1/debian/rules lomiri-system-settings-1.3.1/debian/rules
--- lomiri-system-settings-1.3.1/debian/rules	2024-12-17 13:13:52.000000000 +0100
+++ lomiri-system-settings-1.3.1/debian/rules	2025-06-23 13:21:23.000000000 +0200
@@ -29,9 +29,10 @@
 override_dh_auto_configure:
 	# Debian defines CMAKE_INSTALL_LOCALSTATEDIR as /usr/var, which is wrong.
 	# So until Debian bug 719148 is fixed, do it ourselves.
-	dh_auto_configure -- -DCMAKE_INSTALL_LOCALSTATEDIR="/var"	\
-	                     -DMODERN_PYTHON_DBUSMOCK=ON		\
-	                     -DENABLE_LIBDEVICEINFO=ON			\
+	dh_auto_configure -- -DCMAKE_INSTALL_LOCALSTATEDIR="/var"				\
+	                     -DLOMIRI_KEYBOARD_PLUGIN_PATH="/usr/lib/lomiri-keyboard/plugins"	\
+	                     -DMODERN_PYTHON_DBUSMOCK=ON					\
+	                     -DENABLE_LIBDEVICEINFO=ON						\
 	                     $(CONFIGURE_FLAGS)
 
 override_dh_auto_build:

--- End Message ---
--- Begin Message ---
Unblocked lomiri-system-settings.

--- End Message ---

Reply to: