[0.7] Plugin läuft nicht wenn selbst gebaut

MegaV0lt · 8. November 2022

Hallo,

wollte für das Plugin skinflatplus einen Patch einbauen. Folgendes habe ich gemacht:

Code

sudo apt-get install build-essential devscripts
sudo apt-get build-dep vdr-plugin-skinflatplus
sudo mkdir -p ~/src/vdr-plugin-skinflatplus
cd  ~/src/vdr-plugin-skinflatplus
sudo apt-get source vdr-plugin-skinflatplus
cd vdr-plugin-skinflatplus

patch -p1 < ../patchtvs.diff

sudo dch -l local "added tvs infos"
sudo dpkg-buildpackage -b -us -uc
sudo dpkg -i ...deb"

Alles anzeigen

VDR schmiert jetzt ab:

Code

Nov 08 10:05:15 vdr01 vdr[9022]: [9022] starting plugin: skinflatplus
Nov 08 10:05:15 vdr01 vdr[9022]: [9022] skinflatplus: TrueColor OSD found
Nov 08 10:05:15 vdr01 vdr[9022]: [9022] skinflatplus: load theme:
Nov 08 10:05:15 vdr01 vdr[9022]: Magick: abort due to signal 11 (SIGSEGV) "Segmentation Fault"...
Nov 08 10:05:15 vdr01 systemd[1]: vdr.service: Main process exited, code=killed, status=6/ABRT

Hier der Patch:

Diff

diff --git a/HISTORY b/HISTORY
index 487870dc..5c14a36b 100644
--- a/HISTORY
+++ b/HISTORY
@@ -1,7 +1,8 @@
 VDR Plugin 'skinflatplus' Revision History
 ---------------------------------------
-2021-XX-XX: Version X.X.X
+2022-XX-XX: Version X.X.X
 - [add] option for widget dvbdevices to display device numbers userfriendly
+- [add] display additional information from TVScraper (causedBy and reason)
 
 2021-02-24: Version 0.6.1
 - [fix] '%' sign not shown in menu weather widget at certian font
diff --git a/displaymenu.c b/displaymenu.c
index 2c4c4b01..df7a3097 100644
--- a/displaymenu.c
+++ b/displaymenu.c
@@ -3214,7 +3214,6 @@ void cFlatDisplayMenu::DrawItemExtraRecording(const cRecording *Recording, cStri
       if (recInfo->Aux()) {
         string str_epgsearch = xml_substring(recInfo->Aux(), "<epgsearch>", "</epgsearch>");
         string channel, searchtimer, pattern;
-
         if (!str_epgsearch.empty()) {
           channel = xml_substring(str_epgsearch, "<channel>", "</channel>");
           searchtimer = xml_substring(str_epgsearch, "<searchtimer>", "</searchtimer>");
@@ -3222,19 +3221,30 @@ void cFlatDisplayMenu::DrawItemExtraRecording(const cRecording *Recording, cStri
             searchtimer = xml_substring(str_epgsearch, "<Search timer>", "</Search timer>");
         }
 
+        string str_tvscraper = xml_substring(recInfo->Aux(), "<tvscraper>", "</tvscraper>");
+        string causedby, reason;
+        if (!str_tvscraper.empty()) {
+          causedby = xml_substring(str_tvscraper, "<causedBy>", "</causedBy>");
+          reason = xml_substring(str_tvscraper, "<reason>", "</reason>");
+        }
+
         string str_vdradmin = xml_substring(recInfo->Aux(), "<vdradmin-am>", "</vdradmin-am>");
         if (!str_vdradmin.empty()) {
           pattern = xml_substring(str_vdradmin, "<pattern>", "</pattern>");
         }
 
-        if ((!channel.empty() && !searchtimer.empty()) || !pattern.empty()) {
-          text << endl << endl << tr("additional information") << ":" << endl;
+        if ((!channel.empty() && !searchtimer.empty()) || (!causedby.empty() && !reason.empty()) || !pattern.empty()) {
+          text << endl << endl << tr("additional information") << ":";  // Zusatzinfos anzeigen
           if (!channel.empty() && !searchtimer.empty()) {
-            text << "EPGsearch: " << tr("channel") << ": " << channel << ", " << tr("search pattern") << ": "
+            text << endl << "EPGsearch: " << tr("channel") << ": " << channel << ", " << tr("search pattern") << ": "
                  << searchtimer;
           }
+          if (!causedby.empty() && !reason.empty()) {  // TVScraper
+            text << endl << "TVScraper: " << tr("caused by") << ": " << causedby << ", " << tr("reason") << ": "
+                 << reason;
+          }
           if (!pattern.empty()) {
-            text << "VDRadmin-AM: " << tr("search pattern") << ": " << pattern;
+            text << endl << "VDRadmin-AM: " << tr("search pattern") << ": " << pattern;
           }
         }
       }
@@ -3601,7 +3611,6 @@ void cFlatDisplayMenu::SetRecording(const cRecording *Recording) {
     if (recInfo->Aux()) {
       string str_epgsearch = xml_substring(recInfo->Aux(), "<epgsearch>", "</epgsearch>");
       string channel, searchtimer, pattern;
-
       if (!str_epgsearch.empty()) {
         channel = xml_substring(str_epgsearch, "<channel>", "</channel>");
         searchtimer = xml_substring(str_epgsearch, "<searchtimer>", "</searchtimer>");
@@ -3609,19 +3618,29 @@ void cFlatDisplayMenu::SetRecording(const cRecording *Recording) {
           searchtimer = xml_substring(str_epgsearch, "<Search timer>", "</Search timer>");
       }
 
+      string str_tvscraper = xml_substring(recInfo->Aux(), "<tvscraper>", "</tvscraper>");
+      string causedby, reason;
+      if (!str_tvscraper.empty()) {
+        causedby = xml_substring(str_tvscraper, "<causedBy>", "</causedBy>");
+        reason = xml_substring(str_tvscraper, "<reason>", "</reason>");
+      }
+
       string str_vdradmin = xml_substring(recInfo->Aux(), "<vdradmin-am>", "</vdradmin-am>");
       if (!str_vdradmin.empty()) {
         pattern = xml_substring(str_vdradmin, "<pattern>", "</pattern>");
       }
 
-      if ((!channel.empty() && !searchtimer.empty()) || !pattern.empty()) {
-        recAdditional << endl;
-        if (!channel.empty() && !searchtimer.empty()) {
-          recAdditional << "EPGsearch: " << tr("channel") << ": " << channel << ", " << tr("search pattern") << ": "
+      if ((!channel.empty() && !searchtimer.empty()) || (!causedby.empty() && !reason.empty()) || !pattern.empty()) {
+        if (!channel.empty() && !searchtimer.empty()) {  // EPGSearch
+          recAdditional << endl << "EPGsearch: " << tr("channel") << ": " << channel << ", " << tr("search pattern") << ": "
                         << searchtimer;
         }
-        if (!pattern.empty()) {
-          recAdditional << "VDRadmin-AM: " << tr("search pattern") << ": " << pattern;
+        if (!causedby.empty() && !reason.empty()) {  // TVScraper
+          recAdditional << endl << "TVScraper: " << tr("caused by") << ": " << causedby << ", " << tr("reason") << ": "
+                        << reason;
+        }
+        if (!pattern.empty()) {  // VDR-Admin
+          recAdditional << endl << "VDRadmin-AM: " << tr("search pattern") << ": " << pattern;
         }
       }
     }
@@ -4695,7 +4714,7 @@ int cFlatDisplayMenu::DrawMainMenuWidgetDVBDevices(int wLeft, int wWidth, int Co
       }
     }
     channelName = strDevice.str().c_str();
-    
+
     cString str = cString::sprintf("%d", i + 1);  // Display Tuners 1..4
     if (Config.MainMenuWidgetDVBDevicesNativeNumbering) {
        str = cString::sprintf("%d", i);  // Display Tuners 0..3
diff --git a/po/de_DE.po b/po/de_DE.po
index ae3eedce..e2cb4fee 100644
--- a/po/de_DE.po
+++ b/po/de_DE.po
@@ -132,6 +132,12 @@ msgstr "Kanal"
 msgid "search pattern"
 msgstr "Suchmuster"
 
+msgid "caused by"
+msgstr "Ausgelöst durch"
+
+msgid "reason"
+msgstr "Grund"
+
 msgid "Recording information"
 msgstr "Aufnahme Informationen"

Alles anzeigen

Dachte nicht, dass der so schwerwiegende Folgen hat...

Wie bekomme ich das wieder hin.

Und wie komme ich zurück zum alten Plugin aus dem Repo?

seahawk1986 · 8. November 2022

Zitat von MegaV0lt

Dachte nicht, dass der so schwerwiegende Folgen hat...

Da hilft nur sich einen Backtrace anzeigen zu lassen, um zu sehen, wo es genau knallt. Ich vermute stark, dass die Logzeile von Magick nur ein Folgefehler ist.

Zitat von MegaV0lt

Und wie komme ich zurück zum alten Plugin aus dem Repo?

Einfach das Paket erneut installieren: sudo apt install --reinstall vdr-plugin-skinflatplus

MegaV0lt · 8. November 2022

Vielen Dank!

Irgendwie klappt das aber nicht ganz

Code

darkwing@vdr01:~/src/skin-flatplus/vdr-plugin-skinflatplus-0.6.1+git20220607-29-361e958e$ sudo apt install --reinstall vdr-plugin-skinflatplus
Paketlisten werden gelesen... Fertig
Abhängigkeitsbaum wird aufgebaut.       
Statusinformationen werden eingelesen.... Fertig
Erneute Installation von vdr-plugin-skinflatplus ist nicht möglich,
es kann nicht heruntergeladen werden.
0 aktualisiert, 0 neu installiert, 0 zu entfernen und 0 nicht aktualisiert.

seahawk1986 · 8. November 2022

Welche Paketquellen nutzt du denn?

apt policy vdr vdr-plugin-skinflatplus

MegaV0lt · 8. November 2022

Sorry. Hab jetzt ein apt remove gefolgt von apt install gemacht. Das Original-Plugin ist nun wieder aktiv

Was mich aber wurmt ist, dass der Patch anscheinend das Plugin zerstört... Ich komme da schon mal nicht weiter

MegaV0lt · 9. November 2022

Gibt es denn einen unterschied wie das Plugin gebaut wird im Repo <-> Lokal?

seahawk1986 · 9. November 2022

Das kommt darauf an, wie man es genau baut... - eine pbuilder-Umgebung oder ähnliches (daneben gibt es noch sbuild, whalebuilder und ein paar andere) ist da oft praktisch, weil man die Build-Dependencies reproduzierbar aus vorgegebenen Quellen holen kann.

Erst mal würde ich schauen, ob die VDR-Header zur installierten VDR-Version passen (die sollten beide aus dem selben Quellpaket gebaut worden sein): apt policy vdr vdr-dev

kamel5 · 9. November 2022

MegaV0lt ,

nur so zur Info:

Dein Patch funktioniert hier ohne Probleme und zeigt im Menü auch das an, was es soll.

Ich würde allerdings in den obigen Zeilen 36 und 40 jeweils ein "<< endl" vor dem Semikolon einfügen, sonst wird alles in eine Zeile geschrieben.

Grüße

kamel5

MegaV0lt · 9. November 2022

Gut zu wissen... Kannst Du einen Screenshot machen?

Ich wollte es so machen wie bei EPGSearch...

seahawk1986

Hab das auf dem VDR gebaut so wie oben beschrieben:

Code

sudo apt-get install build-essential devscripts
sudo apt-get build-dep vdr-plugin-skinflatplus
sudo mkdir -p ~/src/vdr-plugin-skinflatplus
cd  ~/src/vdr-plugin-skinflatplus
sudo apt-get source vdr-plugin-skinflatplus
cd vdr-plugin-skinflatplus
patch -p1 < ../patchtvs.diff
sudo dch -l local "added tvs infos"
sudo dpkg-buildpackage -b -us -uc
sudo dpkg -i ...deb"

MegaV0lt · 9. November 2022

seahawk1986

Hier noch die gewünschten Infos

Code

darkwing@vdr01:~$ apt policy vdr vdr-dev
vdr:
  Installiert:           2.6.1-0yavdr0~focal
  Installationskandidat: 2.6.1-0yavdr0~focal
  Versionstabelle:
 *** 2.6.1-0yavdr0~focal 500
        500 http://ppa.launchpad.net/seahawk1986-hotmail/vdr-2.6.1/ubuntu focal/main amd64 Packages
        100 /var/lib/dpkg/status
     2.4.1-4ubuntu1 500
        500 http://de.archive.ubuntu.com/ubuntu focal/universe amd64 Packages
vdr-dev:
  Installiert:           2.6.1-0yavdr0~focal
  Installationskandidat: 2.6.1-0yavdr0~focal
  Versionstabelle:
 *** 2.6.1-0yavdr0~focal 500
        500 http://ppa.launchpad.net/seahawk1986-hotmail/vdr-2.6.1/ubuntu focal/main amd64 Packages
        100 /var/lib/dpkg/status
     2.4.1-4ubuntu1 500
        500 http://de.archive.ubuntu.com/ubuntu focal/universe amd64 Packages

Alles anzeigen

kamel5 · 9. November 2022

Zitat von MegaV0lt

Ich wollte es so machen wie bei EPGSearch...

Ist ja soweit OK. Ohne das endl wird es aber in eine Zeile geschrieben und dann sieht es nicht so schön aus.

Grüße

kamel5

MegaV0lt · 10. November 2022

Vielen Dank für den Screenshot. Ich wusste nicht, dass die Informationen von EPGSearch und TVScraper gleichzeitig auftreten können.

Ich habe den Patch jetzt so geändert, dass das endl immer vor den entsprechenden Zeilen kommt. Damit müsste es immer passen.

Leider kann ich das Plugin nicht selber bauen und testen Beim bauen von epgd hat mein vorgehen von oben immer funktioniert

kamel5 · 10. November 2022

Zitat von MegaV0lt

Ich wusste nicht, dass die Informationen von EPGSearch und TVScraper gleichzeitig auftreten können.

Ich habe das zum Testen nur händisch in eine info-Datei eingetragen. Ich würde aber trotzdem nicht ausschließen, das da Mehreres auftreten kann. Es gibt ja auch noch VDRadmin-AM...

Grüße

kamel5

MegaV0lt · 10. November 2022

OK. Ich habe im Quelltext noch eine Szelle gefunden, wo die Infos abgefragt werden. Den Patch habe ich auch schon aktualisiert. Habe aber keine Ahnung wozu das ist. Ein Test wäre gut; oder vielleicht kennt sich ja jemand besser aus

kamel5 · 10. November 2022

Zitat von MegaV0lt

Habe aber keine Ahnung wozu das ist.

Das scheint für die Info-Anzeige zu sein (Blaue Taste).

Da fehlen auch noch die "endl".

Grüße

kamel5

MegaV0lt · 10. November 2022

Vielen Dank fürs testen. Habe die endl eingefügt. Müsste passen. Patch oben

kamel5 · 10. November 2022

Grüße

kamel5

MegaV0lt · 10. November 2022

Hier mal die Ausgaben vom Bauen:
https://www.dropbox.com/s/59ff…3y/build_skinflatplus.log

kamel5

Hast Du zufällig auch yaVDR laufen?

Oder jemand anderes?

seahawk1986

Kann ich den Patch ins GIT machen? Was passiert wenn es dann wieder nicht läuft?

kamel5 · 10. November 2022

Zitat von MegaV0lt

Hast Du zufällig auch yaVDR laufen?

Leider nicht. Ich übersetze immer alles selber.

Grüße

kamel5

seahawk1986 · 11. November 2022

Zitat von MegaV0lt

Kann ich den Patch ins GIT machen? Was passiert wenn es dann wieder nicht läuft?

Probiers - ich konnte erfolgreich ein Paket mit dem Patch lokal bauen und den VDR danach erfolgreich starten - die Build-Warnungen sollte sich noch jemand ansehen, der mehr Routine mit C++ hat.

[0.7] Plugin läuft nicht wenn selbst gebaut

Jetzt mitmachen!

Teilen