livebuffer patch für vdr 1.7.16 (aus rmm svn)

M-Reimer · December 14, 2010 at 7:40 AM

Quote

Original von IG88
sowohl vdr als auch rmm-vdr sind keine community projekte, demokratie gibts da (fast) nicht

Beim rmm-vdr: Volle Zustimmung. Der ist einzig und alleine als Firmware für die Reel-Box gedacht und wie Reel diese erstellt, ist deren Sache.

Beim VDR dagegen läuft das mit dem "Mitarbeiten" nicht so schlecht, wie das einem immer erzählt wird. Ich habe selber schon Patches für den VDR erstellt und auch übernommen bekommen. Das kls die Erweiterung der Plugin-Schnittstelle für eine Funktion, die er nicht direkt im VDR haben will, ablehnt, glaube ich erst, wenn ich es gesehen habe.

Und was "Community-Projekte" angeht: Wenn du darunter "alles von der Community wird unbesehen übernommen" meinst, dann kann ich dir sagen, dass das nicht funktionieren wird. Irgendwer (kann auch eine Gruppe sein) muss Patches prüfen, bewerten und ggf. (evtl. mit Änderungswünschen) auch ablehnen. Wie schonmal geschrieben: Wenn kls nicht regulierend eingreifen würde, wäre VDR heute nicht von so guter Qualität.

Ja, das war wieder Off-Topic, aber das musste ich noch loswerden. Weitere Diskussion zum Plugin bitter hier hin: http://www.vdrportal.de/board/thread.php?threadid=91664 und zu den anderen Themen bitte ggf. Threads aufmachen und hier verlinken. Danke.

KlausiHH · December 14, 2010 at 2:30 PM

Quote

Original von Torsten73
ich stimme dem ebenfalls zu.

... viele Argumente ...

...denen ich mal uneingeschränkt zustimmen möchte.

Es mir absolut unbegreiflich, dass vielen die Funktionalität des livebuffer patches scheinbar vollkommen unnütz erscheint.

Da leider bei mir auch nicht mal das normale Timeshift funktioniert [1] - ich bin immer am überlegen wieder zu einem alten VDR zurück zu gehen, würde dann aber doch vieles tolle an yaVDR 0.3 vermissen -, wäre ich noch viel mehr an diesem Patch interessiert.

Ich würde mich sehr freuen, wäre unheimlich dankbar, wenn mir jemand kurz erklären könnte, wie ich den Patch aus diesem Thread selber mal einbauen kann.
Dazu bräuchte ich vermutlich irgendwelche Quellen, die dann gepatcht werden. Das ganze neu kompilieren, und gut? Wie ginge das genau mit yaVDR 0.3?

Tausend Dank im Voraus.

lg
JPM

[1] Mein vdr hält bei Pause das Bild zwar an, hängt sich aber gleichzeitig
auf. Nichts geht mehr. Davon berichten im yaVDR-Unterforum auch manche andere.

carel · December 14, 2010 at 2:40 PM

Hallo, hier auch ein Livebuffer fan (und sicher auch meine Töchter...)

Quote

Sobald ich allerdings "Pause" drück stürzt der VDR mit folgendem Syslog-Eintrag ab:

vdr[3440]: segfault at 0 ip 002107a0 sp bfa31638 error 4 in libc-2.11.1.so[19d000+153000]

Dieser Fehler tritt bei mir nur auf in zusammenhang mit text2skin.
Hab ich endlich mein Livebuffer, muss ich wieder auf dem anthraskin verzichten.

Carel

PS: zweite vdr: idem dito: LB lauft sauber ohne text2skin

Toxic-Tonic · December 14, 2010 at 2:45 PM

Copperhead

Sag mal, hast du schon ein Patch, der zu deinem ExtPatchNG passt?

Habe zwar die Rejects aufgelöst (denke ich), aber compilieren tut es trotzdem nicht!

Gruß

Toxic

M-Reimer · December 14, 2010 at 2:53 PM

Quote

Original von KlausiHH
Es mir absolut unbegreiflich, dass vielen die Funktionalität des livebuffer patches scheinbar vollkommen unnütz erscheint.

Ebenso geht es denjenigen, die den Sinn einfach nicht finden können (mich eingeschlossen). Ich stimme an der Stelle mit kls überein. Wer das aber ausführlicher diskutieren will: Bitte Thread aufmachen!

carel · December 14, 2010 at 2:59 PM

Quote

aber compilieren tut es trotzdem nicht!

@Toxic: welcher fehler hast du dann?

Ich habe livebuffer.o fest ins OBJ list der Makefile aufgenommen
Nur dann compilierte er, irgendwo will der Makefile der 'USE_LIVEBUFFER' in Make.config nicht lesen.

Copperhead · December 14, 2010 at 3:02 PM

Nein hab ich nicht. Momentan ist mir in dem Thread hier noch zu viel los. Das gibt eine ewige Rennerei, wenn ich den jetzigen Patch inden Extensionpatch einbaue.

DKVT · December 14, 2010 at 7:38 PM

Quote

Original von carel

Dieser Fehler tritt bei mir nur auf in zusammenhang mit text2skin.

Hab es eben auch ohne das text2skin-Plugin getestet und dann läuft der LiveBuffer im yavdr!

Vielleicht können wir ja die "Grundsatzdiskussionen" in einen anderen Thread verschieben und uns hier wieder dem RMM-Patch widmen.

Grundsätzlich scheint es ja zu funktionieren, jetzt müsste man wissen warum es am text2skin-Plugin scheitert. Wie kann man da auf die Suche gehen? Bin jetzt nicht so der Debugging-Experte...

Grüße

Toxic-Tonic · December 14, 2010 at 7:46 PM

Quote

Original von Copperhead
Nein hab ich nicht. Momentan ist mir in dem Thread hier noch zu viel los. Das gibt eine ewige Rennerei, wenn ich den jetzigen Patch inden Extensionpatch einbaue.

Muss ja nicht eingebaut sein, nur angepasst werden, damit er auch gegen einen gepatchten VDR läuft. Dachte wenn du eh schon testest...

Gruß

Toxic

IG88 · December 15, 2010 at 12:00 AM

hi,

das mit dem kein comunity projekt nicht in den falsch hals bekommen, das war nur eine feststellung - ich habe nichts dazu gesagt wie es besser ginge
ich dachte da im vergleich dazu nur an sowas wie den dvb treiber unter linuxtv.org, da gibts svn/hg/git und es arbeiten mehrere leute daran, man kann sich für schreibrechte im code qualifizieren und es gibt einen plan, "ofizielle" ziele, ... - das das aber auch mal nicht so klapt konnte man beim thema dvb-s2 sehen

bei vdr weis man einfach nie wann es wie weiter geht, ist halt eine one man show und das merkt man dann auch bei der entwicklungsgeschwindigkeit (dafür ist selbst die entwicklerversion zuverlässig in der nutzung), einer allein in seiner wenigen freizeit schafft nun mal nicht so viel und es ist für kls auch nicht immer leicht dinge zu implementieren von denen er selbst nicht 100%ig überzeugt ist (da fallen mit utf8 und ts ein), hat letztlich viel neuen code bei wenig neuen funktionen bedeutet

carel · December 15, 2010 at 10:41 AM

Quote

jetzt müsste man wissen warum es am text2skin-Plugin scheitert

Dazu mal ein backtrace erstellt:

Code

Core was generated by `./vdr --user=carel --userdump --config=/etc/vdr --video=/files/VDR-opnamen --lo'.
Program terminated with signal 11, Segmentation fault.
#0  __strlen_sse2 () at ../sysdeps/x86_64/multiarch/../strlen.S:31
31      ../sysdeps/x86_64/multiarch/../strlen.S: Bestand of map bestaat niet.
        in ../sysdeps/x86_64/multiarch/../strlen.S
(gdb) bt
#0  __strlen_sse2 () at ../sysdeps/x86_64/multiarch/../strlen.S:31
#1  0x00007fe2f9932973 in cText2SkinStatus::Replaying (this=0x7fe2f9b5e780, Name=0x0, FileName=0xb7a990 "/files/VDR-opnamen/LiveBuffer", On=true)
    at status.c:45
#2  0x00000000004d5e3d in cStatus::MsgReplaying (Control=0xb736b0, Name=0x0, FileName=0xb7a990 "/files/VDR-opnamen/LiveBuffer", On=<value optimized out>)
    at status.c:47
#3  0x000000000049bfb0 in cReplayControl (this=0xb736b0) at menu.c:4718
#4  0x000000000049c19d in cRecordControls::PauseLiveVideo () at menu.c:4559
#5  0x00000000004e9afa in main (argc=<value optimized out>, argv=<value optimized out>) at vdr.c:1091
Current language:  auto
The current source language is "auto; currently asm".
(gdb) quit

Display More

Und text2skin letzter debug meldungen:

Code

parsing: {MenuItem}
pushing to parent
parsing: {MenuCurrent}
pushing to parent
rotating parent to object
cText2SkinStatus::Replaying((null))

Aber ob das alles reicht und ob jemand mit Kenntnis sich damit beschäftigen will, ich bezweifele es...

Carel

gda · December 15, 2010 at 10:55 AM

Quote

Original von carel

Code

Core was generated by `./vdr --user=carel --userdump #1  0x00007fe2f9932973 in cText2SkinStatus::Replaying (this=0x7fe2f9b5e780, Name=0x0, FileName=0xb7a990 "/files/VDR-opnamen/LiveBuffer", On=true)
    at status.c:45

Ohne allzu tief rein gesehen zu haben, denke ich, es liegt daran, dass die Aufnahme im Lifebuffer keinen Namen hat, siehe Name=0x0. Dann kracht strlen(). Der Name müsste zumindest mit einem Leerstring initialisiert werden.

Gerald

carel · December 15, 2010 at 11:07 AM

Txs Gerald!

Dass muss zu lösen sein, bin leider kein c Guru, aber wer weiß wie weit ich komme...

KlausiHH · December 15, 2010 at 11:27 AM

Quote

Ich würde mich sehr freuen, wäre unheimlich dankbar, wenn mir jemand kurz erklären könnte, wie ich den Patch aus diesem Thread selber mal einbauen kann.
Dazu bräuchte ich vermutlich irgendwelche Quellen, die dann gepatcht werden. Das ganze neu kompilieren, und gut? Wie ginge das genau mit yaVDR 0.3?

Schade. Dann muss ich wohl auf was "offizielles" warten.

Ich hätte das gern mal selber ausprobiert. Ein ganz blutiger Anfänger bin
ich nun auch nicht, weiß aber ja nicht mal, wo ich anfangen sollte, wo ich
dazu was nachlesen oder selber lernen könnte.

hotzenplotz5 · December 15, 2010 at 11:53 AM

in yavdr wird es reinkommen wenn es abschaltbar ist, und wenn es funktioniert.

ob als patch oder plugin ist mir scheissegal, vdr ist doch eh ein flickwerk aus (bei yavdr) ca. 40 patches plus die sourcen von vdr

gda · December 15, 2010 at 11:57 AM

Quote

Original von KlausiHH

Schade. Dann muss ich wohl auf was "offizielles" warten.

Das Problem ist, dass das mal eben kurz nicht zu erklären ist. Ich finde das ziemlich viel Arbeit. Ich versuche es trotzdem und wenn es klappt dann ist es dein Job das ins Wiki zu kippen. Ich mache das komplett aus dem Kopf, wenn was nicht klappt etwas nachdenken, oder aufgeben. Keine Rückfragen!

Code

# patch downloaden:
wget <patch url>
sudo apt-get update
apt-get source vdr
cd vdr-1.7.16
#letzten patch in debian/patches/00list suchen
#neuer name für livebuffer patch analog zu den Beispielen in der 00list ausdenken
dpatch-edit-patch patch <neuer name für livebuffer patch ohne extension> <letzter patch ohne extension>
patch -p1 <~/<downgeloadeter patch>
#eventuelle rejects auflösen
#danach:
rm -f *.orig *.rejs
exit
#neuer name für patch ohne extension an debian/patches/00list anhängen
#was sinnvolles in die changelog schreiben und revision erhöhen:
dch -i
#neuen Patch akzeptieren
debian/rules accept-patches
#Paket bauen
dpkg-buildpackage

Display More

Gerald

DKVT · December 15, 2010 at 12:33 PM

gda

Dein Tipp war goldrichtig! Hab jetzt auf die Schnelle in der status.c des Text2skin-Plugins die Zeilen 49 - 86 auskommentiert und schon klappt es.

Wenn ich heute Abend mehr Zeit habe werde ich versuchen hier einen Patch anzuhängen der die Stringlänge beachtet.

carel · December 15, 2010 at 12:38 PM

In status.c zeile 43:

Code

if (On) {

ändern in

Code

if (On && Name != NULL) {

reicht, oder ?

carel · December 15, 2010 at 12:50 PM

Hotzenplotz

Quote

in yavdr wird es reinkommen wenn es abschaltbar ist

IMHO ist das der Fall, man kann eben das originale verhalten wählen!
Im Aufnahmen Menü gibts bei Pausieren eine dritte option: Livebuffer
Im setup.conf: PauseKeyHandling = 3

Carel

gda · December 15, 2010 at 1:02 PM

Quote

Original von DKVT
gda

Dein Tipp war goldrichtig! Hab jetzt auf die Schnelle in der status.c des Text2skin-Plugins die Zeilen 49 - 86 auskommentiert und schon klappt es.

Wenn ich heute Abend mehr Zeit habe werde ich versuchen hier einen Patch anzuhängen der die Stringlänge beachtet.

Sorry, aber das finde ich ziemlich schlecht. Nur für einen Patch des VDRs jetzt noch Plugins patchen? Ihr wisst doch gar nicht wo das sonst überall krachen könnte. Lieber im Livebuffer-Patch dafür sorgen, dass der Name initialisiert wird. So möchte ich das lieber nicht in yaVDR sehen.

Gerald

Participate now!