[EPGSearch] Neue Version v2.4.5

Quote from TomJoad

Neuer allgemeiner Parameter für Suchtimer: Frist zur Begrenzung der Timererstellung

Vorneweg: Die Implementierung tut, was sie soll; und 999 Tage (gut 2,7 Jahre) als Defaultwert sind sicherlich mehr, als ein EPG jemals liefern dürfte.

Dennoch finde ich die Implementierung nicht optimal, weil sie mit Konventionen bricht:

• In EPGsearch steht der Wert 0 als Defaultwert bei praktisch allen Einstellungen dafür, dass eine Funktion abgeschaltet ist. Das ist hier aber weder der Fall noch überhaupt möglich.
• In den Menüs haben wir an vielen Stellen die Einheiten, anstatt sie in den Fließtext einzubetten, in eckigen Klammern angefügt, damit sie leichter gesehen werden. Auch bei den Einstellungen oberhalb der Timer-Zeitbegrenzung ist das beispielsweise der Fall.
• In den Menüs haben wir entweder auf Verben verzichtet bzw. meist die Infinitiv-Formulierung ("Per OSD warnen") anstelle der imperativen Form ("Warne per OSD") genutzt.
• In den Man-Pages habe ich bei der Überarbeitung, wo immer möglich, die exakte Schreibweise des Menüeintrages als Überschrift bzw. Kennzeichnung für die entsprechende Einstellung verwendet, damit eine Zuordnung ohne großes Rätselraten gegeben ist.

Der folgende Patch (jetzt ohne Tippfehlerteufel):

• 0001a Revised timer limit.patch.txt

… würde die obigen Punkte adressieren:

Quote

Zeitbegrenzung für Timer [t]

Begrenzt die Erzeugung von Timern auf einen Zeitraum in Tagen. Ein Wert von 0 deaktiviert die Begrenzung.

Kurzer Test: Baut und startet ohne Fehlermeldung.

Auf einem 64Bit-System funktioniert diese epgsearch-Version 2.4.5 bei mir wunderbar. Auf einem 32Bit-System crashed die aber, nachdem ich Debian bookworm auf trixie upgegraded habe.

Der angehaengte Patch fixed diesen Crash.

Ich wuerde mich freuen, wenn der Patch in das offizielle Repo uebernommen wuerde.

Gruss
S:oren

Hi TomJoad,

ich benutze epgsearch schon lange auf meinem Aufnahme-Server. Würde es aber ,wegen der Bedienung, lieber auf meinem Client nutzen. Das scheitert aber daran, dass epgsearch die Timer immer lokal anlegt, unabhängig von der VDR eigenen Client/Server Lösung (svdrpverbindung nur mit Standard Maschine). Ist das Verhalten eventuell änderbar? Vermutlich würde dann dann die Timerprüfung nicht mehr funktionieren, aber darauf könnte ich ggfls verzichten.

Ansonsten super. dass Du dich dem Plugin angenommen hast und es mit dem prima Plugin epgsearch weitergeht.

Quote from TomJoad

machtnix epgsearch kennt genausolang wie der vdr remote Timer. Wenn das eingerichtet ist, gibt es bei "Timer editieren" die Abfrage "Aufnehmen auf ...". Es ist auch der Konfliktcheck remote einrichtbar. Nur Suchtimer sind auf lokale timer beschränkt, weil die Überwachung mit timersdone und epgsearchdone.data auf Wiederholungen und korrekte Aufnahmen nicht mehr leicht realisierbar ist.

Remote Timer aber nur mit Plugin vdr-plugin-remotetimers, oder?

Quote from TomJoad

was hältst du davon, wie vdr mit %jd und cast auf (intmax_t) zu arbeiten?

Ergibt ja Sinn, das genauso zu machen wie im vdr. Danke fuer den Hinweis. Angepasster Patch anbei.

Gruss,
S:oren

Quote from TomJoad

Nur Suchtimer sind auf lokale timer beschränkt, weil die Überwachung mit timersdone und epgsearchdone.data auf Wiederholungen und korrekte Aufnahmen nicht mehr leicht realisierbar ist.

Schade, gerade die Suchtimer machen das Plugin so nützlich. Dann bleibt wohl doch nur das umständliche einstellen der Suchtimer mit dem remoteosd-plugin

Hallo

Wäre da nicht ein Modus "übertrage alle Suchtimer auf Server und gleiche dort nur alles lokal ab" , wie wenn dort direkt erstellt möglich?

MfG Stefan

Quote from machtnix

Schade, gerade die Suchtimer machen das Plugin so nützlich. Dann bleibt wohl doch nur das umständliche einstellen der Suchtimer mit dem remoteosd-plugin

Geht auch mit live

Hm, was ist denn msprintf() nun schon wieder!?

Der Compiler (gcc 14.2) beschwert sich jedenfalls nicht, und es crashed auch nichts. Aber ich kann es gerne , wenn gewuenscht, auch noch anpassen und testen, ob sich wenigstens nichts verschlechtert.

Gruss
S:oren

Quote from TomJoad

Patch scheint ziemlich vollständig zu sein, aber m.E. fehlt noch in searchtimer_thread.c::SummaryExtended()

start und stop bei msprintf()

Falls das wirklich auch angepasst werden muss, dann vermutlich auch in

const char *cPendingNotification::ToText(void)
const char *cRecDone::ToText(void)
const char *cNoAnnounce::ToText(void)

Gruss
S:oren

Ich denke, ich habe eine Moeglichkeit gefunden, alle auftretenden time_t zu checken (siehe commit message). C ist eben auch nicht meine Muttersprache. Aber wieder 'was gelernt.

So richtig verstehe ich den Sinn dieses msprintf() nicht, man haette ja durchgaengig cString nehmen koennen. Ist vermutlich ein Artefakt aus alten Zeiten.

Hoffentlich passt damit jetzt alles. Laeuft bei mir auf 32bit und 64bit.

Gruss
S:oren