[dbus] Was, wie, wo?

MegaV0lt · February 23, 2025 at 8:46 AM

Von dbus hab ich null Ahnung. Ich weis nur, dass VDR wohl damit zu steuern geht und dass es ein Plugin dbus2vdr gibt.

https://github.com/yavdr/vdr-plugin-dbus2vdr

Das Plugin wird aber nicht mehr weiterentwickelt.

Meine eigentliche Frage ist, ob man über dbus (ohne Plugin) am VDR Meldungen (Info, Warning oder Error) ausgeben kann?

Mit dem Plugin geht nur Info (mtInfo)

Eine Dokumentation zum VDR dbus konnte ich nicht finden

seahawk1986 · February 23, 2025 at 10:01 AM

Soweit ich weiß war damals das primäre Ziel die svdrpsend MESG "message" Funktion des VDR nachzubauen, weil der VDR damals noch nicht mit mehreren SVDRP-Clients gleichzeitig umgehen konnte und so jedes Skript, das eine Nachricht absetzen wollte da selber dafür hätte sorgen müssen, das erneut zu versuchen.

Da svdrpsend MESG auch nur mtInfo nutzt, war das gut genug.

Man könnte das Interface von dbus2vdr um einen neuen Endpunkt erweitern, so dass man da auch noch andere Werte als mtInfo angeben kann - da scheint es ja insgesamt vier zu geben:

Code: https://git.tvdr.de/?p=vdr.git;a=blob;f=skins.h;h=15c729f2c2389944b7762467a69f8ea3c2e8d1de;hb=HEAD#l37

enum eMessageType { mtStatus = 0, mtInfo, mtWarning, mtError }; // will be used to calculate color offsets!

Generell ist DBus eine Schnittstelle zur Kommunikation zwischen Prozessen. Es kann neben dem System-Bus (den das dbus2vdr-Plugin standardmäßig nutzt) auch noch weitere Instanzen geben, die für z.B. die Sitzung eines Benutzers existieren. Eine Anwendung dafür sind Benachrichtigungsdienste: https://wiki.ubuntuusers.de/Benachrichtigungsdienst/ - das Arch Linux Wiki hat da eine Liste von Standalone-Programmen, die man dafür nutzen könnte: https://wiki.archlinux.org/title/Desktop_…ions#Standalone - das läuft dann halt über den X-Server (was nicht alle Ausgabeplugins abdeckt), der VDR selber hat kein DBus Interface.

seahawk1986 · February 23, 2025 at 11:59 AM

Der VDR filtert Messages mit mtStatus raus, also bleiben drei Möglichkeiten - hier mal ein Patch für dbus2vdr, der das nachrüstet:

Diff: dbus2vdr-send-message.diff

Index: vdr-plugin-dbus2vdr-20211130143000experimental/README
===================================================================
--- vdr-plugin-dbus2vdr-20211130143000experimental.orig/README  2025-02-23 11:13:17.503710831 +0100
+++ vdr-plugin-dbus2vdr-20211130143000experimental/README       2025-02-23 11:54:45.345967607 +0100
@@ -352,9 +352,12 @@

 Interface "skin"
 ----------------
-- display a message at the OSD
+- display a message on the OSD
   vdr-dbus-send.sh /Skin skin.QueueMessage string:'message text'

+- display a message with a certain type (represented with different colors by VDR) on the OSD
+  vdr-dbus-send.sh /Skin skin.SendMessage string:'message text' string:'info|warning|error'
+
 - get information about available skins
   vdr-dbus-send.sh /Skin skin.ListSkins
   You will get an array of index, name and description.
Index: vdr-plugin-dbus2vdr-20211130143000experimental/skin.c
===================================================================
--- vdr-plugin-dbus2vdr-20211130143000experimental.orig/skin.c  2025-02-23 11:13:17.503710831 +0100
+++ vdr-plugin-dbus2vdr-20211130143000experimental/skin.c       2025-02-23 11:57:52.518634884 +0100
@@ -11,7 +11,7 @@
     "<!DOCTYPE node PUBLIC \"-//freedesktop//DTD D-BUS Object Introspection 1.0//EN\"\n"
     "       \"http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd\">\n"
     "<node>\n"
-    "  <interface name=\""DBUS_VDR_SKIN_INTERFACE"\">\n"
+    "  <interface name=\"" DBUS_VDR_SKIN_INTERFACE "\">\n"
     "    <method name=\"CurrentSkin\">\n"
     "      <arg name=\"replycode\"    type=\"i\" direction=\"out\"/>\n"
     "      <arg name=\"skin\"         type=\"(iss)\" direction=\"out\"/>\n"
@@ -30,6 +30,12 @@
     "      <arg name=\"replycode\"    type=\"i\" direction=\"out\"/>\n"
     "      <arg name=\"replymessage\" type=\"s\" direction=\"out\"/>\n"
     "    </method>\n"
+    "    <method name=\"SendMessage\">\n"
+    "      <arg name=\"messageText\"  type=\"s\" direction=\"in\"/>\n"
+    "      <arg name=\"messageType\"  type=\"s\" direction=\"in\"/>\n"
+    "      <arg name=\"replycode\"    type=\"i\" direction=\"out\"/>\n"
+    "      <arg name=\"replymessage\" type=\"s\" direction=\"out\"/>\n"
+    "    </method>\n"
     "  </interface>\n"
     "</node>\n";

@@ -108,8 +114,43 @@

     cDBusHelper::SendReply(Invocation, replyCode, replyMessage);
   };
-}

+  static void SendMessage(cDBusObject *Object, GVariant *Parameters, GDBusMethodInvocation *Invocation)
+  {
+    const char *messageText = NULL;
+    const char *messageType = NULL;
+    g_variant_get(Parameters, "(&ss)", &messageText, &messageType);
+
+    gint32 replyCode = 501;
+    cString replyMessage;
+    if ((messageText == NULL) || (*messageText == 0)) {
+       replyMessage = "Missing message";
+       goto end;
+    }
+    if ((messageType == NULL) || (*messageType == 0)) {
+       replyMessage = "Missing message type";
+       goto end;
+    }
+
+    eMessageType mtype;
+    if (strcmp(messageType, "info") == 0) {
+        mtype = mtInfo;
+    } else if (strcmp(messageType, "warning") == 0) {
+        mtype = mtWarning;
+    } else if (strcmp(messageType, "error") == 0) {
+        mtype = mtError;
+    } else {
+        replyMessage = "invalid message type, must be one of info, warning, error";
+        goto end;
+    }
+
+    Skins.QueueMessage(mtype, messageText);
+    replyCode = 250;
+    replyMessage = "Message queued";
+end:
+    cDBusHelper::SendReply(Invocation, replyCode, replyMessage);
+  };
+}

 cDBusSkin::cDBusSkin(void)
 :cDBusObject("/Skin", cDBusSkinHelper::_xmlNodeInfo)
@@ -118,6 +159,7 @@
   AddMethod("ListSkins", cDBusSkinHelper::ListSkins);
   AddMethod("SetSkin", cDBusSkinHelper::SetSkin);
   AddMethod("QueueMessage", cDBusSkinHelper::QueueMessage);
+  AddMethod("SendMessage", cDBusSkinHelper::SendMessage);
 }

 cDBusSkin::~cDBusSkin(void)

Display More

MegaV0lt · February 23, 2025 at 12:36 PM

Vielen Dank für die Erläuterungen.

Geht das nicht direkt mit dbus-send an den VDR so wie in dem Beispiel:

Code

In the bin-directory you find a helper script for sending dbus messages to the vdr.
Instead of calling:
  dbus-send --system --type=method_call --dest=de.tvdr.vdr \
            --print-reply /Plugins/<pluginname> \
            de.tvdr.vdr.plugin.SVDRPCommand string:'command' string:'parameter'

Nur halt mit 'Message' und MessageType?

MegaV0lt · February 23, 2025 at 12:38 PM

Oh, gleich schon ein Patch! Vielen Dank. Das muss ich dann mal testen, wenn ich wieder mal Zeit habe.

Kannst Du einen Pull Request machen? Wird bestimmt übernommen

seahawk1986 · February 23, 2025 at 12:54 PM

Quote from MegaV0lt

Geht das nicht direkt mit dbus-send an den VDR so wie in dem Beispiel:

Klar geht das, das ist dann halt etwas länger als mit dem vdr-dbus-send Skript - mal zum Vergleich:

Code

vdr-dbus-send /Skin skin.SendMessage string:"Das ist eine Info" string:"info"
dbus-send --system --type=method_call --dest=de.tvdr.vdr --print-reply /Skin de.tvdr.vdr.skin.SendMessage string:"Das ist eine Info" string:"info"

mini73 · February 23, 2025 at 9:10 PM

Danke, seahawk1986

Du könntest "info" als Fallback nehmen, falls kein Type angegeben ist. Bin aber zu lange raus, um zu wissen, ob optionale Parameter bei dbus spezifiziert sind...

Lars.

seahawk1986 · February 23, 2025 at 11:10 PM

Quote from mini73

Du könntest "info" als Fallback nehmen, falls kein Type angegeben ist. Bin aber zu lange raus, um zu wissen, ob optionale Parameter bei dbus spezifiziert sind...

DBus ist da ziemlich unflexibel, https://dbus.freedesktop.org/doc/dbus-api-d…#nullable-types listet da ein paar Möglichkeiten, aber ich sehe da leider keinen Weg das API-kompatibel zu QueueMessage zu gestalten, da kann man sich höchstens mit Variants oder Arrays austoben oder einen leeren String mitschicken, wie das z.B. bei https://github.com/yavdr/vdr-plug…ter/README#L165 nötig ist, wenn man nur ein Argument übergeben will:

Code

$ svdrpsend plug softhddevice STAT
220 yavdr-noble-demo SVDRP VideoDiskRecorder 2.7.3; Sun Feb 23 23:09:18 2025; UTF-8
910 SuspendMode is NOT_SUSPENDED
221 yavdr-noble-demo closing connection
$ vdr-dbus-send /Plugins/softhddevice plugin.SVDRPCommand string:'STAT' string:''
method return time=1740348300.260114 sender=:1.141 -> destination=:1.237 serial=3882 reply_serial=2
   int32 910
   string "SuspendMode is NOT_SUSPENDED"

MegaV0lt · February 24, 2025 at 3:16 PM

Quote from seahawk1986

dbus-send --system --type=method_call --dest=de.tvdr.vdr --print-reply /Skin de.tvdr.vdr.skin.SendMessage string:"Das ist eine Info" string:"info"

Funktioniert leider nicht. Hab auch QueueMessage probiert:

Code

dbus-send --system --type=method_call --dest=de.tvdr.vdr --print-reply /Skin de.tvdr.vdr.skin.SendMessage string:"message" string:"info"
Error org.freedesktop.DBus.Error.UnknownMethod: No such method “SendMessage”



dbus-send --system --type=method_call --dest=de.tvdr.vdr --print-reply /Skin de.tvdr.vdr.skin.QueueMessage string:"Das ist eine Info" string:"info"
Error org.freedesktop.DBus.Error.InvalidArgs: Type of message, “(ss)”, does not match expected type “(s)”

seahawk1986 · February 24, 2025 at 4:43 PM

Sicher, dass du den VDR nach Bau und Installation des gepatchten Plugins neu gestartet hast?

MegaV0lt · February 24, 2025 at 4:45 PM

Das Plugin hab ich noch nicht probiert. Das war der Versuch das über dbus-send zu machen. Ohne Plugin

MegaV0lt · February 24, 2025 at 4:56 PM

Das Plugin funktioniert mit dem Patch:

Code

darkwing@vdr01:~/src/dbus2vdr$ vdr-dbus-send /Skin skin.SendMessage string:"Das ist eine Info" string:"info"
method return time=1740412536.652426 sender=:1.107 -> destination=:1.108 serial=28 reply_serial=2
   int32 250
   string "Message queued"
darkwing@vdr01:~/src/dbus2vdr$ vdr-dbus-send /Skin skin.SendMessage string:"Das ist eine Info" string:"warning"
method return time=1740412543.781398 sender=:1.107 -> destination=:1.109 serial=29 reply_serial=2
   int32 250
   string "Message queued"
darkwing@vdr01:~/src/dbus2vdr$ vdr-dbus-send /Skin skin.SendMessage string:"Das ist eine Info" string:"error"
method return time=1740412551.841852 sender=:1.107 -> destination=:1.110 serial=36 reply_serial=2
   int32 250
   string "Message queued"

Display More

MegaV0lt · February 25, 2025 at 2:38 PM

Quote from mini73

"info" als Fallback nehmen, falls kein Type angegeben ist

Hab mal so was eingebaut: https://github.com/MegaV0lt/vdr-p…b548bffebc38482

Setzt 'info' als Typ, wenn der String leer is'

Participate now!