Bigpatch für VDR 1.3.30

Hi,
Erstmal ein grosses Danke an Frank fuer seine tolle Arbeit.
Allerdings bin ich der Meinung dass einige Patches unnoetig sind, vor allem solche die sich durch Plugins oder eine passende runvdr erschlagen lassen.
Dazu gehoert z.B. da Anpassen der Lautstaerke und des letzten Senders.
Ich hab dies in meiner Distri extra im Setup-Plugin integriert und nun funktioniert dies nicht mehr, da der dazugehoerige Patch meine Einstellungen "vernichtet". Daher wuerde ich dafuer plaedieren Patches welche "sauberer" geloest werden koennen aussen vor zu lassen.

Zitat

Original von helau
Allerdings bin ich der Meinung dass einige Patches unnoetig sind, vor allem solche die sich durch Plugins oder eine passende runvdr erschlagen lassen.
Dazu gehoert z.B. da Anpassen der Lautstaerke und des letzten Senders.
Ich hab dies in meiner Distri extra im Setup-Plugin integriert und nun funktioniert dies nicht mehr, da der dazugehoerige Patch meine Einstellungen "vernichtet". Daher wuerde ich dafuer plaedieren Patches welche "sauberer" geloest werden koennen aussen vor zu lassen.

Gerade dieses Feature finde ich _sehr_ sinnvoll!

Warum benutzt du das Setup-Plugin bitte? Und was soll daran sauberer sein als mit dem Patch?

Hallo,
Jedes Plugin ist sauberer als ein Patch. Aber in dem vorliegenden Falle waere mir schon sehr geholfen, wenn die StartupLautstaerke per default disabled waere:
+ StartupVolume = 256;
anstelle von
+ StartupVolume = MAXVOLUME;
wuerde dies beheben.
Die Moeglichkeit Plugins zu erstellen wurde aus dem Grunde geschaffen, dass man VDR nicht wegen jedem kleinen Feature patchen muss. Und falls man nun sich die Muehe macht ein Plugin zu erstellen ist es nicht gerade huebsch wenn dessen Moeglichkeiten durch einen Patch lahmgelegt werden.
Generell hab ich ja gar nichts dagegen, aber man sollte die Default Werte sauber setzen, so dass der Patch wenn moeglich nichts kaputt macht.

Zitat

Original von Franzose

Gerade dieses Feature finde ich _sehr_ sinnvoll!

Warum benutzt du das Setup-Plugin bitte? Und was soll daran sauberer sein als mit dem Patch?

Kann mich Franzose nur anschließen - absolut sinnvolles Feature

Gruß Christian

Zitat

Original von helau
Jedes Plugin ist sauberer als ein Patch.

Das wage ich zu bezweifeln. Ob der experimentelle Code nun im Plugin oder im Hauptprogramm ist, macht eigentlich keinen Unterschied. Bei einem Fehler wird es halt nicht laufen, aber das trifft auf beide zu.

Die Plugins wurden IMHO ja nur wegen der besseren Benutzerfreundlichkeit erstellt...

/edit: Außerdem hast nicht auf meine Frage geantwortet: Wozu brauchst du das Setup-Plugin?

Hi,

natürlich ist es möglich das es nicht am Live-Buffer Patch liegt, aber da der wohl zuletzt eingefügt wurde habe ich darauf getippt.
Anscheinend hat aber sonst niemand das Problem bei abgeschaltetem Live-Buffer Patch. Ich werds heute noch mal testen...

Die Hardware kann man zwar nie ausschließen aber der Rechner läuft in dieser Konfiguration schon über ein Jahr ohne Probleme. Das Staub wird regelmäßig entfernt, dafür wird der Rechner aber nicht bewegt. Der letzte Sommer war von den Temperaturen wohl eher kritisch, aber da gab es keine Probleme.

Hi,
Erstmal ist das Feature natuerlich sinnnvoll, sonst haette ich es nicht in mein Admin Plugin eingebaut ( Sorry nicht Setup ). Aber es ist nicht sinnvoll, dass der Patch per default die Einstellungen des Plugins zunichte macht.

Zitat

Das wage ich zu bezweifeln. Ob der experimentelle Code nun im Plugin oder im Hauptprogramm ist, macht eigentlich keinen Unterschied. Bei einem Fehler wird es halt nicht laufen, aber das trifft auf beide zu.

Deine Einstellung zu Plugins scheint aber nicht gerade die Beste zu sein Wenn ich das weiterdenke haette man nie Plugins benoetigt sondern haette saemtliche Features auch als Patches zur Verfiuegung stellen koennen ...

Hi,

nur mal so zwischendurch. Ich nutze auch das Setup Plugin und finde die damit erstellte Menüstruktur wesentlich besser als alles im Hauptmenü zu haben ist aber geschmackssache.

Hallo zusammen,

ein dickes Lob an alle, die dieses großartige Stück möglich gemacht haben. Der 1.3.30 mit BP läuft hervorragend, hab noch nix feststellen können. Durch das Einkommentieren der zwei Stellen in der vdr.c stürzt VDR auch nicht mehr bei der Lautstärkenänderung im Wiedergabemodus ab. Perfekt!

Also vielen Dank und weiter so...

(Muss jetzt nur nochmal sehen, dass ich das burn-Plugin ordentlich zum Laufen bekomme, unter 1.3.29 lief das bei mir nicht so gut. Aber leider sind immer alle Festplatten der Welt zu klein und ich hab nicht mal mehr 10 Gigabyte, um ein Image zu erstellen... Und mit aufräumen konnte ich mich nie wirklich gut anfreunden... :D)

Zitat

Original von helau
Deine Einstellung zu Plugins scheint aber nicht gerade die Beste zu sein Wenn ich das weiterdenke haette man nie Plugins benoetigt sondern haette saemtliche Features auch als Patches zur Verfiuegung stellen koennen ...

Nein, gerade nicht. Du hast mich da nicht verstanden! Plugins sind allein zur Funktionserweiterung (DVD, mp3, femon etc. pp.), also Features, die im VDR _nicht_ enthalten sind.

Patches sind dazu da, Funktionen, die im VDR bereits enthalten sind bzw. deren Einstellungsmöglichkeiten zu erweitern und zu erreichen, dass bei sinnvollen Änderungen (wie bei dieser hier), die Funktion in den Vanilla-VDR aufgenommen wird.

Das ist eine ganz klare Trennung. Daher hat diese Audiofunktion auch nichts im Plugin, sondern im Patch zu suchen meiner Meinung nach.

Außerdem hast du zum zweiten Mal nicht auf meine Frage geantwortet: Wozu brauchst du das Setup-Plugin?

Moin,
Franzose les doch bitte richtig, Ich hab geschrieben dass es sich nicht ums Setup sondern Admin Plugin handelt. Und das habe ich genau fuer solche Sachen entwickelt.
Ich geb Dir uebrigens durchaus recht, dass diese Funktion in VDR gehoert, aber wenn schon dann sollte der default Wert das Grundverhalten von VDR nicht aendern und somit eine evtl vorhandene andere Loesung blockieren.

Zitat

Original von helau
Franzose les doch bitte richtig, Ich hab geschrieben dass es sich nicht ums Setup sondern Admin Plugin handelt. Und das habe ich genau fuer solche Sachen entwickelt.

Setup/Admin...du weißt doch, was ich meine. Aber bitte gut, wenn du nicht auf meine Fragen/Anregungen eingehen willst, dann ist das dein Problem.

Meiner Meinung nach, ist diese Audiofunktion im Patch bestens aufgehoben und wenn du Entwickler bist, wird es ja auch ein leichtes für dich sein, die entsprechende Stelle im Code zu beheben.

Zitat

Setup/Admin...du weißt doch, was ich meine. Aber bitte gut, wenn du nicht auf meine Fragen/Anregungen eingehen willst, dann ist das dein Problem.

Weisst Du ueberhaupt was das Admin Plugin macht ? Ich bezweifle es stark, denn sonst wuerdest Du die Frage nicht stellen.

Zitat

Meiner Meinung nach, ist diese Audiofunktion im Patch bestens aufgehoben und wenn du Entwickler bist, wird es ja auch ein leichtes für dich sein, die entsprechende Stelle im Code zu beheben.

Auch darin geb ich Dir Recht aber das kann ja nicht der Sinn der Sache sein, dass jeder rumpatched wo es ihm Spass macht - auch wenns ne Menge Spass macht

P.S. Jetzt reichts aber mit der Diskussion Es ging mir doch eigentlich nur darum dass die Zeile:
+ StartupVolume = MAXVOLUME;
in
+ StartupVolume = 256;
geaendert wird, und da hat doch sicher niemand was degegen

Zitat

Original von helau

Weisst Du ueberhaupt was das Admin Plugin macht ? Ich bezweifle es stark, denn sonst wuerdest Du die Frage nicht stellen.

Nein, weiß ich nicht, wozu sollte ich DICH sonst fragen, WOZU du dieses Plugin brauchst? Aber scheinbar willst du die Frage nicht verstehen...

Zitat

Original von helauAuch darin geb ich Dir Recht aber das kann ja nicht der Sinn der Sache sein, dass jeder rumpatched wo es ihm Spass macht - auch wenns ne Menge Spass macht smile

Gut, dass du mir zustimmst, dass die Audiofunktion besser im Patch aufgehoben ist. Aber wo ist dann das Problem? Nimm' die Funktion aus deinem Plugin heraus und verwende die vom Bigpatch...und schon ist kein Problem mehr vorhanden.

Zitat

Original von Franzose
Nein, weiß ich nicht, wozu sollte ich DICH sonst fragen, WOZU du dieses Plugin brauchst? Aber scheinbar willst du die Frage nicht verstehen...

http://www.vdr-wiki.de/wiki/index.php/Admin-plugin
P.S. Ich habe ne eigene Distri - ohne solch ein Setup Tool macht sich das ziemlich schlecht

Hallo,

franzose
helau

ich muss dem Franzosen recht geben. Ich finde auch das solche Sachen wie
Startlautstaerke und Startkanal zu den Grunfunktionen des VDRs gehoeren sollten (also im Patch) und nicht in ein Plugin ausgelagert werden sollten.

Ich will das Setup/Admin Plugin nicht einsetzen, da ich nicht zu den 'staendig was an der Konfiguration' Gurus gehoere und zudem noch andere Personen den VDR bedienen und nicht mit Sonderfunktionen belastet werden sollen.

Trozdem wers braucht kann ja benutzen.

Nichts fuer Ungut
Gruss Leo

Zitat

Original von Scorp
@all

Also bei mir tritt wieder das Problem mit dem programmieren von Timern über TVOnscreen auf --> Neustart!

[...]

Hi Frank,

ist bei mir leider auch wieder so. Log auch bei mir nicht besonders aussagekräftig (ausser vielleicht die alloc_pages Meldung vom Kernel (?))

Aug 23 17:49:11 vdr vdr[3995]: Text2Skin: menu display update thread started (pid=3995, tid=327695)
Aug 23 17:49:11 vdr vdr[3995]: Text2Skin: menu display update thread ended (pid=3995, tid=327695)
Aug 23 17:49:11 vdr vdr[3642]: tvonscreen magazine::Show()
Aug 23 17:49:20 vdr vdr[3642]: too many different colors used in palette
Aug 23 17:49:35 vdr last message repeated 58 times
Aug 23 17:49:37 vdr vdr[4046]: Text2Skin: menu display update thread started (pid=4046, tid=344079)
Aug 23 17:49:37 vdr vdr[4046]: too many different colors used in palette
Aug 23 17:49:37 vdr last message repeated 2 times
Aug 23 17:49:39 vdr vdr[4046]: Text2Skin: menu display update thread ended (pid=4046, tid=344079)
Aug 23 17:49:39 vdr vdr[4057]: Text2Skin: menu display update thread started (pid=4057, tid=360463)
Aug 23 17:49:40 vdr su: pam_unix2: session finished for user root, service su
Aug 23 17:49:50 vdr kernel: __alloc_pages: 7-order allocation failed (gfp=0x1f0/0)
Aug 23 17:49:50 vdr kernel: DVB: registering new adapter (Siemens/Technotrend/Hauppauge PCI rev1.3).
Aug 23 17:49:50 vdr kernel: DVB: registering new adapter (TT-Budget/WinTV-NOVA-CI PCI).

Nachtrag:
Habe es nochmal versucht, jetzt kam zumindest die Einstellungsseite des OSD für den Timer und schmiert danach ab; timer wird nicht angelegt...

Aug 23 18:25:41 vdr vdr[10832]: Text2Skin: menu display update thread ended (pid=10832, tid=229389)
Aug 23 18:25:41 vdr vdr[10715]: tvonscreen magazine::Show()
Aug 23 18:25:47 vdr vdr[10715]: too many different colors used in palette
Aug 23 18:25:53 vdr last message repeated 17 times
Aug 23 18:25:56 vdr vdr[10863]: Text2Skin: menu display update thread started (pid=10863, tid=245773)
Aug 23 18:25:58 vdr vdr[10863]: Text2Skin: menu display update thread ended (pid=10863, tid=245773)
Aug 23 18:25:58 vdr vdr[10864]: Text2Skin: menu display update thread started (pid=10864, tid=262157)
Aug 23 18:26:06 vdr vdr[10715]: timer 6 (32 1916-2006 'ZDF spezial Flut in den Alpen') added (active)
Aug 23 18:26:06 vdr vdr[10864]: Text2Skin: menu display update thread ended (pid=10864, tid=262157)
Aug 23 18:26:06 vdr su: pam_unix2: session finished for user root, service su
Aug 23 18:26:15 vdr kernel: __alloc_pages: 7-order allocation failed (gfp=0x1f0/0)
Aug 23 18:26:15 vdr kernel: DVB: registering new adapter (Siemens/Technotrend/Hauppauge PCI rev1.3).
Aug 23 18:26:15 vdr kernel: DVB: registering new adapter (TT-Budget/WinTV-NOVA-CI PCI).

Gruß,
Marcus

helau
Wollt Dir eigentlich nee PM schicken....ging irgendwie nicht
Nettes Plugin,
aber bist Du Dir sicher das Du Deinen PC per ASPI-Wakeup aufwachen lassen willst. siehe Screenshot WIKI

Hi,
habe den aktuellen Bigpatch mal wieder zum testen installiert. Jedoch habe ich das gleiche problem, wie ich es beim letzten teszen mit dem Bigpatch hatte.
Ich verwende Text2Skin (CVS ca. 1 Monat alt) mit dem Enigma-Skin. Wenn ich jetzt nach de patchen VDR starte und den Enigma Skin mit der gleichen Größe, wie in der ungepatchten Version verwenden will, meckert VDR, dass der Sin zu groß sei. Hab den Sin auch mal 10% kleiner probiert, aber immer noch der gleiche Fehler.

Ändert der Bigpatch irgendwas an Text2Skin oder dem Skin, dass er dann zu groß für den Speicher ist (ungemoddete Karte/ 2 MB. Oder ist das ein anderer Fehler, mach ich was Falsch?

MfG
Karsten