Posts by kfb77

Danke, Funktion passt wieder.

Mein Compiler bringt aber noch ein paar Warnings:

g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++17 -Wfatal-errors -Wundef -Wno-deprecated-declarations -Wno-overloaded-virtual -Wno-unused-variable -c -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"live"' -DTNTVERSION=30000 -DCXXTOOLVER=30000 -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DHAVE_PCRE2 -I/usr/src/vdr/vdr/include -I/usr/src/vdr/vdr/include -I.. edit_timer.cpp
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++17 -Wfatal-errors -Wundef -Wno-deprecated-declarations -Wno-overloaded-virtual -Wno-unused-variable -c -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"live"' -DTNTVERSION=30000 -DCXXTOOLVER=30000 -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DHAVE_PCRE2 -I/usr/src/vdr/vdr/include -I/usr/src/vdr/vdr/include -I.. error.cpp
edit_timer.ecpp: In member function ‘virtual unsigned int {anonymous}::_component_::operator()(tnt::HttpRequest&, tnt::HttpReply&, tnt::QueryParams&)’:
edit_timer.ecpp:149:18: warning: ignoring return value of ‘bool std::operator==(const __cxx11::basic_string<_CharT, _Traits, _Allocator>&, const _CharT*) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]’, declared with attribute ‘nodiscard’ [-Wunused-result]
  149 |       monitoring == "none";  // tvscraper does the monitoring -> no epgsearch monitoring
      |       ~~~~~~~~~~~^~~~~~~~~
In file included from /usr/include/c++/13/string:54,
                 from /usr/include/tnt/compident.h:33,
                 from /usr/include/tnt/component.h:33,
                 from /usr/include/tnt/ecpp.h:33,
                 from edit_timer.cpp:6:
/usr/include/c++/13/bits/basic_string.h:3727:5: note: declared here
 3727 |     operator==(const basic_string<_CharT, _Traits, _Alloc>& __lhs,
      |     ^~~~~~~~
edit_timer.ecpp:151:18: warning: ignoring return value of ‘bool std::operator==(const __cxx11::basic_string<_CharT, _Traits, _Allocator>&, const _CharT*) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]’, declared with attribute ‘nodiscard’ [-Wunused-result]
  151 |       monitoring == "none";  // monitoring only supported for single shot timers
      |       ~~~~~~~~~~~^~~~~~~~~
/usr/include/c++/13/bits/basic_string.h:3727:5: note: declared here
 3727 |     operator==(const basic_string<_CharT, _Traits, _Alloc>& __lhs,
      |     ^~~~~~~~
edit_timer.ecpp:153:47: warning: ignoring return value of ‘bool std::operator==(const __cxx11::basic_string<_CharT, _Traits, _Allocator>&, const _CharT*) [with _CharT = char; _Traits = char_traits<char>; _Alloc = allocator<char>]’, declared with attribute ‘nodiscard’ [-Wunused-result]
  153 |       if (!aux_epgsearch.found() ) monitoring == "none";
      |                                    ~~~~~~~~~~~^~~~~~~~~
/usr/include/c++/13/bits/basic_string.h:3727:5: note: declared here
 3727 |     operator==(const basic_string<_CharT, _Traits, _Alloc>& __lhs,
      |     ^~~~~~~~
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++17 -Wfatal-errors -Wundef -Wno-deprecated-declarations -c -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"live"' -DTNTVERSION=30000 -DCXXTOOLVER=30000 -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DHAVE_PCRE2 -I/usr/src/vdr/vdr/include ffmpeg.cpp
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++17 -Wfatal-errors -Wundef -Wno-deprecated-declarations -Wno-overloaded-virtual -Wno-unused-variable -c -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"live"' -DTNTVERSION=30000 -DCXXTOOLVER=30000 -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DDISABLE_TEMPLATES_COLLIDING_WITH_STL -DHAVE_PCRE2 -I/usr/src/vdr/vdr/include -I/usr/src/vdr/vdr/include -I.. tooltip.cpp

Ja, sollte es, wenn auch --log2rec angegeben wurde.

Die Parameter "online" und "after" sind nur für den internen Aufruf vom markad Plugin gedacht, nicht für den Aufruf per Command Line. Vielleicht besser, ich entferne die mal aus der man Page.

Lasse mal --online weg und ersetzte "after" mit "-" oder "nice".

Und warum liegt markad bei dir unter sbin ? Da gehört es eigentlich nicht hin.

Edit: ich habe die Codezeile gefunden, wo deine ungültige Parameter Kombination abgefangen wird:

if ((bAfter) && (config.online)) return EXIT_SUCCESS;

Ich ändere das mal in einen Fehler Return Code und in eine Fehlermeldung.

Edit2:

Quote from nobanzai

Das passiert scheinbar seit 2024-06.

Das liegt dann aber nicht an einer Änderung von markad.

Da ist die Zeile im Original Code von vor 7 Jahren, den ich übernommen habe. Die gleiche Prüfung war da auch schon drin, nur ein Variablen Name hat sich geändert.

Der letzte Commit hat ein Problem: "Verzeichnis" beim Timer anlegen funktioniert nicht mehr. Timer werden grundsätzlich ohne angegebenes Verzeichnis erstellt. Ein revert vom letzten Commit beseitigt das Problem.

Habe ich das richtig verstanden:

- Wenn ich einen Timer mit Überwachung "Sendungskennung" (also Event ID) oder "Uhrzeit" anlege, sorgt epgsearch aufgrund des AUX Feldes im Timer dafür, dass die Zeiten des Timers bei Änderung des Events verschoben werden, auch ohne dass es einen Suchtimer dafür gibt ? VDR arbeitet bei "nicht VPS" Timer grundsätzlich immer nach der Uhrzeit ?

- Was passiert, wenn sich die Event ID ändert ? Erfolgt dann keine Aufnahme oder macht epgsearch einen Fallback zur Uhrzeit bei der Suche und aktualisiert die Event ID ?

- So eine Option macht beim Anlegen eines Suchtimers keinen Sinn, da dies eh Funktion jedes Suchtimers ist ?

Quote from kfb77

Das wird nicht gehen, weil dann die Kontrolle zur Erzeugung des Timers bei epgsearch liegt und nicht bei Live.

Dann ziehe ich diese Aussage auch zurück, dann kann das wohl epgsearch auch.

Quote from shofmann

… die Überwachung per Senderkennung anbietet, gehe ich davon aus, dass er sich auch selbst darum kümmert und Verschiebungen trackt.

Stimmt. Das wusste ich nicht, weil ich keine VDR Oberfläche nutze. VDR ist bei mir Headless Backend im Container, Oberfläche ist Live.

Quote from shofmann

Nach den schlechten Erfahrungen mit VPS bei VCRs benutze ich VPS überhaupt nicht.

Sehe ich auch so. Ich sehe in deiner Plugin Liste markad. Da gibt es eine "VPS light" Funktion, die die VPS Events, falls notwendig, zur Optimierung der Marken heranzieht. Gerade bei ÖR gibt es oft keinen klaren Logo Start/Ende, dafür aber gutes VPS.

Quote from nobanzai

Jo, das wäre toll.

Das wird nicht gehen, weil dann die Kontrolle zur Erzeugung des Timers bei epgsearch liegt und nicht bei Live.

Quote from nobanzai

Und bei Sendungskennung tritt die dann anstelle der Uhrzeit`?

Das neue Feature in Live habe ich mir noch nicht angeschaut, aber so würde ich das auch interpretieren.

MarkusE Wer ändert dann die Uhrzeiten des Timer, wenn der Event sich verschiebt ? Ich nehme mal an, dass muss dann Live machen, weil im VDR kenne ich so ein Feature nicht.

Quote from shofmann

Wobei die VPS-Überwachung gegebenenfalls nur dann gesetzt wird, wenn der entsprechende Sender auch eine VPS-Unterstützung anbietet

So einfach ist das Thema nicht: Jeder mir bekannte Astra FTA unterstützt VPS. Bei den ÖR passt das auch ganz gut mit Start/Ende der Sendung zusammen. Manchmal ist da nach dem Ende noch ein Preview drin, aber ganz OK. Die Privaten senden den VPS Start zum geplanten Termin (egal, wann die Sendung tatsächlich startet) und keinen VPS Ende.

Ein VPS Timer wird nicht "überwacht", der wird getriggert durch die VPS Events. Der zugehörige Event leitet sich aus der geplanten Startzeit ab. Die bleibt immer gleich, auch wenn sich die tatsächliche Startzeit verschiebt. Somit darf hier epgserach den Timer nicht anpassen, falls sich das EPG ändert.

Ein normaler Timer startet zu seinen definierten Zeiten. Wenn sich die Uhrzeit des EPG Eintrag für Timer ändert, verschiebt epgsearch den Timer.

Um epgsearch die Info mitzugeben, ob es einen VPS Timer oder einen normalen Timer erzeugen soll.

Die Halloween Version 4.2.5 von vdr-plugin-markad ist verfügbar.

Bei Probleme bitte immer die vollständige markad.log posten.

2024-10-31: Version 4.2.5
- plugin: remove use of MainThreadHook()
- fix crash from missing codec
- detect black screen despite permanent news ticker
- some minor bug fixes and optimizations, see git

Setzte die Warnings innerhalb #ifdef <irgendwas> und in plugins.c machst du vor der Stelle einen #undef <irgendwas> und danach wieder einen #define ...

Quote from wmautner

Aber bei Einstellung "danach" in den Plugin-Einstellungen läßt sich konfigurieren, ob markad während einer anderen Aufnahme oder Wiedergabe laufen soll.

Da geht es aber um Aufnahmen oder Wiedergaben vom VDR, nicht um bereits laufende markad Prozesse. Also kein Problem.

Wenn mein Verdacht von hier stimmt, geht es bei markad mit einem Einzeiler:

diff --git a/plugin/menu.cpp b/plugin/menu.cpp
index 7dde48ec3..2ff397d8d 100644
--- a/plugin/menu.cpp
+++ b/plugin/menu.cpp
@@ -84,6 +84,7 @@ bool cMenuMarkAd::write() {
     Clear();

     bool header=false;
+    status->Check();     // refresh list of running markad tasks
     struct sRecording *Entry=nullptr;
     status->ResetActPos();
     do {

Beim shutdown request wird eh nochmals geprüft, und dann könnte auch bei mir MainThreadHook entfallen.

Kann mir mal jemand, der das markad Plugin nutzt (ich selbst nicht, ich starte markad aus einem Sript raus), wozu das Plugin wissen muss, welche markad Prozesse gerade noch laufen ?

Ich habe im Code nur zwei Stellen gefunden: Im Setup für den Markad Status und beim Shutdown vom VDR um, falls das so konfiguriert ist, diesen zu verhindern.

Edit: und noch: svdrpsend plug markad status

Hintergrund der Frage ist dieser Thread. Falls ich nichts übersehen habe, gibt es keine Notwendigkeit das im MainThreadHook laufend zu aktualisieren, dann reicht das auch an o.g. Stellen.

Quote from kls

Kann markad nicht selber feststellen, ob seine Prozesse fertig sind? Wozu braucht es da MainThreadHook()?

Du fragst mich jetzt Dinge, die Jahre länger im Code drin sind, als ich mich mit markad beschäftigt habe. Und am Plugin habe ich eh fast nichts geändert.

Wenn ich mir den Code so anschaue, brauche ich Variablen aus dem Plugin (die Prozesstabelle, der gestarteten markad Prozesse). Ich muss die Tabelle ändern können, ohne mit Änderungen aus dem Plugin nach einem Recording() Aufruf (Start oder Ende einer Aufnahme) in Konflikt zu kommen. Und die Prüfung der laufenden Prozesse müsste regelmäßig aufgerufen werden. Den Aufwand, was funktionierendes zu ändern, würde ich mir gerne ersparen.

Zumindest eines kenne ich: Das markad Plugin nutzt den MainThreadHook um daraus zur prüfen, ob gestartete markad Prozesse noch laufen, oder inzwischen fertig sind. Da dies nichts mit VDR Strukturen zu tun hat, helfen hier auch keine Locks.

Quote from pierre2000

d.h., jetzt fehlt nur noch sudo apt-get upgrade, oder?

Ja genau

Quote from pierre2000

Kann ich auch nur einzelne Kandidaten auswählen? wenn ich das update für softhdcuvid sehe, bekomme ich schon wieder Bauchschmerzen

Nein, es muss zusammen passen, du kannst nicht ein altes Plugin und einen neuen VDR mischen.

Don't panic; Das ist keine neue Version, nur neu gebaut gegen den neuen VDR, sollte also kein Unterschied machen:

vdr-plugin-softhdcuvid/noble 3.27.3+git20241022-341-6768bc5-0yavdr0~noble amd64 [aktualisierbar von: 3.27.3+git20241022-341-6768bc5-0yavdr0~noble]

Genau so sieht der apt-cache policy aus, aber:

Quote from pierre2000

Habe "sudo apt-get update" und nen reboot gemacht,

Das verstehe ich nicht.

Da ist immer noch die 2.6.7 im Cache.

Mache nochmal sudo apt-get update, ändert sich danach was bei apt-cache policy vdr ?

Ist in ubuntu.sources irgendwas von yavdr drin ?

Quote from pierre2000

eigentlich sollte hier doch auch vdr 2.6.7 aufgelistet werden, oder?

Nein, warum auch ? Die Quellen habe wir doch oben gelöscht, weil du ja die Quellen von VDR 2.7.3 haben wolltest. Oder ???

Edit: OK, du meinst als Update zu 2.7.3, dann ja.

Was sagt denn "apt-cache policy vdr"

und was steht in der seahawk1986-hotmail-ubuntu-vdr-2_7_3-noble.sources drin ?