DVDSwitch: Image von DVD erstellen

Jafar · 3. Februar 2014

Hallo zusammen,

ich habe gerade einen neuen VDR im yaVDR 0.5 augebaut. Die Kiste läuft mittlwerweile recht gut.
Gestern wollte ich zum ersten Mal mit dem DVDSwitch-Plugin ein Image von einer DVD ziehen (war eine gebrannte Fernsehaufnahmen von einem bekannten, also kein DECSS).
Der VDR ist dann quasi abgestürzt und sofort neu gestartet. Eine Image hat der nicht erzeugt, aber auch keine verwertbaren Logeinträge, die auf die Ursache schließen lassen.
In der plugin.dvdswitch.config ist ein debug-Log konfiguriert, in dem aber nichts drin stand. Im syslog sieht man daß er DVDRead startet und gleich wieder stoppt, aber ohne einen Error oder einen Hinweis warum.

Das Plugin läuft an sich. Ich habe diveres isos und VIDEO_TS, die anstandslos in der Übersicht angezeigt werden (Pfad zu den DVD-Verzeichnissen paßt also) und auch abgespielt werden.
Ich habe mir dann das dvdswitch-readscript mal vorgenommen, ein dvdswitch-myreadscript, um ein paar Debug-Ausgaben erweitert und in der plugin.dvdswitch.config eingestellt. Leider scheint er die Konfiguration aus der plugin.dvdswitch.config nicht zu übernehmen. Wenn ich mein Script auf der Kommandozeile ausführe schreibt er meine Debug-Ausgaben in

/var/log/vdr/readdvd.log

und erstellt ein Image, aber über das Plugin selber funktioniert der Aufruf nicht. Es gibt keinen Eintrag unter
/var/log/vdr/readdvd.log.

Was kann ich noch tun, um der Ursache auf den Grund zu kommen? Mein Eindruck ist, daß die Parameter aus der plugin.dvdswitch.config gar nicht gezogen werden. Woher bekommt das plugin die dann?

Gruß Jafar

mini73 · 3. Februar 2014

Die Parameter aus der plugin.<Name>.conf werden nur beim Start des vdr ausgewertet.
Also nach Änderungen entweder Rechner neustarten oder zumindest "sudo restart vdr".

Evtl. willst du noch vdr-dbg und die -dbg-Pakete deiner Plugins (soweit vorhanden) installieren. Dann sollten unter /var/log/vdr crashlog-Dateien mit einem Backtrace entstehen.

Lars.

Jafar · 3. Februar 2014

Zitat von mini73

Die Parameter aus der plugin..conf werden nur beim Start des vdr ausgewertet.
Also nach Änderungen entweder Rechner neustarten oder zumindest "sudo restart vdr".

Das hatte ich gemacht. Einmal per Fernbendienung über das Menü (System -> Einstellung -> Befehle -> Neustart oder so ähnlich) und einmal über einen Rechner Neustart. Alles ohne Auswirkung.
Wird über einen Neustart mit der Fernbedienung nicht auch die config neu geladen?

Zitat von mini73

Evtl. willst du noch vdr-dbg und die -dbg-Pakete deiner Plugins (soweit vorhanden) installieren. Dann sollten unter /var/log/vdr crashlog-Dateien mit einem Backtrace entstehen.

Lars.

Das werde ich heute Abend mal ausprobieren.
Gruß Jafar

mini73 · 3. Februar 2014

Zitat von Jafar

Das hatte ich gemacht. Einmal per Fernbendienung über das Menü (System -> Einstellung -> Befehle -> Neustart oder so ähnlich) und einmal über einen Rechner Neustart. Alles ohne Auswirkung.
Wird über einen Neustart mit der Fernbedienung nicht auch die config neu geladen?

Doch, das sollte beides funktionieren.
Ansonsten kannst du über "ps" nachsehen, wie die Parameter an die Kommandozeile übergeben werden.
Wie sieht die Datei denn aus?

Lars.

Jafar · 3. Februar 2014

Zitat von mini73

Doch, das sollte beides funktionieren.
Ansonsten kannst du über "ps" nachsehen, wie die Parameter an die Kommandozeile übergeben werden.
Wie sieht die Datei denn aus?

Lars.

Hier die plugins.dvdswitch.conf:

Code

#
# Command line parameters for vdr-plugin-dvdswitch
#
#


# Write Debug-Info's in LOGFILE
# (logging is disabled by default)
#
--debug=/tmp/dvdswitch.log


# Use this script reading DVD's into ISO image
# (default is /usr/share/vdr-plugin-dvdswitch/dvdswitch_readdvd)
#
#--readscript=/usr/share/vdr-plugin-dvdswitch/dvdswitch_myreaddvd


# Use this script for writing DVD images onto a DVD
# (default is /usr/share/vdr-plugin-dvdswitch/dvdswitch_writedvd)
#
# --writescript=SCRIPT
#


# Path to DVD images
# (default is /var/lib/video/film/dvd)
#
#--imagedir=/srv/vdr/video.00/iso

Alles anzeigen

Interessanterweise funktioniert das Verzeichnismanagement auch nicht korrekt: Neue Verzeichnisse werden nicht angelegt und bestehende können nicht geändert werden. Bei der Neuanlage wird einfach das Verzeichnis nicht angelegt und bei der Änderung kommt die Meldung, daß das Verzeichnis bereits existieren würde.
Ich habe die Rechte im Ordner /srv/vdr/video.00/iso komplett auf rwx für alle eingerichtet. An den Berechtigungen kann es also nicht liegen.
Eine iso-Image aus einer Aufzeichung zu erzeugen, klappt wunderbar und kann mit DVDSwitch auch abgespielt werden.

Wie kann ich mit ps die Kommandozeilenparameter für die plugins sehen? Lediglich unter dem vdr-User sehe ich die Parameter vom vdr selber. Die Plugins inkl. Parameter sehe ich dort nicht.

Ich habe mal vdr-dbg installiert. Für das Plugin vdr-plugins-dvdswitch selber gibt es wohl kein -dbg, zumindest nicht unter dem naheliegenden Namen.

Das Crashlog bringt leider auch keine brauchbaren Informationen:

Code

Crashlog from Mon Feb  3 18:48:05 CET 2014
----------------------------------


Environment details:
TERM=linux
EXIT_SIGNAL=ABRT
PATH=/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin
PWD=/
JOB=vdr
RESULT=failed
SHLVL=1
PROCESS=main
UPSTART_INSTANCE=
UPSTART_EVENTS=stopped
UPSTART_JOB=vdr-dbg
INSTANCE=
_=/usr/bin/env


Backtrace:
----------

Alles anzeigen

Was kann ich jetzt noch machen?

Gruß Jafar

mini73 · 3. Februar 2014

Dein readscript wird nicht benutzt, weil der Parameter nicht aktiviert ist. Du musst auch das Kommentarzeichen # am Anfang entfernen.

Lars

Jafar · 3. Februar 2014

Zitat von mini73

Dein readscript wird nicht benutzt, weil der Parameter nicht aktiviert ist. Du musst auch das Kommentarzeichen # am Anfang entfernen.

Lars

Ich habe sowohl mit als auch ohne das Kommentarzeichen den selben Fehler und keine Ausgabe im debuglog oder an welcher Stelle auch immer. Von daher habe ich ja meine Zweifel, ob die config überhaupt ausgwertet wird. Es wird ja noch nicht einmal eine
/tmp/dvdswitch.log

angelegt. Ich habe jetzt eben mal spaßeshalber ein existierendes iso-Image auf DVD gebrannt. Funktioniert einwandfrei, ebenfalls keine Ausgab in
/tmp/dvdswitch.log

Gruß Jafar

Jafar · 18. Februar 2014

Ich habe mich mal durch den Quellcode (Version 0.2.2) durchgewühlt. Irgendwie scheint das plugin nicht wirklich komplett konsitent zu sein.
Den in der plugins.dvdswitch.conf aufgeführten --debug Parameter (alternativ -D) gibt es in dem plugin überhaupt nicht. Kein Wunder, daß es keine debug-Ausgabe in das vermeintlich eingestellte Logfile gibt.
Die Read- und Burnskripte an sich funktionieren (also außerhalb des Plugins).
Das Burnskript läßt sich auch aus dem Plugin heraus erfolgreich starten.
Beim Readskripte habe ich den Eindruck, daß er über die Datei und Verzeichniseingabe gar nicht hinaus kommt. Ich habe mal über den Verzeichnismanager in dem Plugin versucht neue Unterordner anzulegen oder bestehende zu ändern. Ich kann da zwar nette Ordnernamen einstellen, aber den Namen übernimmt er nicht korrekt in das plugin
Bei Neuanlage bleibt der Name leer und beim Umbenennen bleibt der Name auf dem ursprünglichen Namen bestehen. Es kommt ja auch die Meldung, daß das Verzeichnis bereits existiert.
In der commands.c findet sich in der cCMDDir::New(void) folgender Code:

Code

...
State = csDirNew;
	SetCols(5);
	SetTitle(CurrentDir());
	SetDir();


	cMainMenuItem *mItem = (cMainMenuItem*)First();
	Ins(new cMenuEditStrItem(tr("New"), Dir, memberof(Dir)),
    	true,
    	mItem);
	while(mItem)
	{
  	mItem->SetSelectable(false);
  	mItem = (cMainMenuItem*)Next(mItem);
	}
	Display();
	cOsdMenu::ProcessKey(kRight);
...
return osContinue;

Alles anzeigen

und in der cCMDDir::New(eKeys Key) später diese Passage:

Code

switch(Key)
  {
	case kOk:
  	if(!isempty(Dir))
  	{
    	char *buffer = NULL;
    	if(0 < asprintf(&buffer, "%s/%s", CurrentDir(), stripspace(Dir))) {
      	dsyslog("dvdswitch: Create new directory: %s", buffer);
      	cFileInfo *info = new cFileInfo(buffer);
      	if(info->isExists())
...

Alles anzeigen

Und hier ist Dir einfach empty, weshalb er dann auch gar keinen Ordner anlegt.
Bei cCMDDir::Edit ist es ähnlich.

Seltsamerweise funktioniert das umbenennen und verschieben von Images ohne Probleme.
Nur im Verzeichnismanager und beim ReadImage verarbeitet er die eingegebenen Ordnernamen nicht weiter.
Ich kann aber auch nicht erkennen woran das liegt.

Hat einer eine Idee?

Hat der Verzeichnismanager überhaupt schon mal funktioniert?

Gruß Jafar

Jafar · 19. Februar 2014

Habe das Problem jetzt in den Griff bekommen. In der commands.c muß im cCMDDir::New(eKeys Key) nur eine Zeile ergänzt werden:

Code

switch(Key)
  {
	case kOk:
//Neue Zeile
        cOsdMenu::ProcessKey(kOk);
//Neue Zeile
        if(!isempty(Dir))
  	{
    	char *buffer = NULL;
    	if(0 < asprintf(&buffer, "%s/%s", CurrentDir(), stripspace(Dir))) {
      	dsyslog("dvdswitch: Create new directory: %s", buffer);
      	cFileInfo *info = new cFileInfo(buffer);
      	if(info->isExists())
...

Alles anzeigen

Für die Edit Funktion gilt das gleiche.

Das Problem beim DVD Image auf die Platte schreiben lag an der realpath Anweisung, die im vdr zu einem Abbruch führt.
Die Lösung habe ich hier gefunden (Stichwort -U_FORTIFY_SOURCE).
Bei der Gelegenheit habe ich das dvdreadskript für mich etwas erweitert, daß es auch die Übergebenen Parameter für den Image-Typ (iso oder VIDEO-TS) auswertet und entsprechend berücksichtigt.
Aus der plugin.dvdswitch.config habe ich den nicht exisitierenden Debug-Parameter gelöscht.
Bei mir läuft jetzt das Plugin. Hat aber eine Weile gedauert

Gruß Jafar

wolfi.m · 21. Februar 2014

Hi Jafar,

erstmal Merci fürs forschen. ..anlegen Verzeichnisse & *.iso erstellen geht jetzt 1A mit DVDswitch.Einen Bug hat die Geschichte noch
beim löschen von *.iso über DVDswitch-Menu kommt es beim verlassen des Menu zum absturtz von VDR - die Ursache -> segfault von SkinnOpacity.
Ob andere Skins auch davon betroffen sind?? ....muss ich noch testen.

Danke nochmals!

Gruss
Wolfgang

wolfi.m · 22. Februar 2014

Hallo,

es wurde nun 6x ein Iso im Menu von DVDswitch gelöscht ...kein Segfault mit SkinnOpacity.Keine Ahnung was dazu führte.
Patch im Anhang!

Gruss
Wolfgang

anbr · 23. Februar 2014

Hi,

wäre nicht folgendes besser ? Das doppelte Aufrufen von cOsdMenu::ProcessKey in den jeweiligen Funktionen kommt mir komisch vor.

Diff

diff --git a/commands.c b/commands.c
index db3c6dc..3360911 100644
--- a/commands.c
+++ b/commands.c
@@ -394,10 +394,11 @@ eOSState cCMDDir::New(void)
 
 eOSState cCMDDir::New(eKeys Key)
 {
+  eOSState state = cOsdMenu::ProcessKey(Key);
   switch(Key)
   {
     case kOk:
-      if(!isempty(Dir))
+      if(state == osUnknown && !isempty(Dir))
       {
         char *buffer = NULL;
         if(0 < asprintf(&buffer, "%s/%s", CurrentDir(), stripspace(Dir))) {
@@ -431,7 +432,7 @@ eOSState cCMDDir::New(eKeys Key)
       break;
   }
 
-  return cOsdMenu::ProcessKey(Key);
+  return state;
 }
 
 
@@ -484,10 +485,11 @@ eOSState cCMDDir::Edit(cMainMenuItem *mItem)
 
 eOSState cCMDDir::Edit(eKeys Key)
 {
+  eOSState state = cOsdMenu::ProcessKey(Key);
   switch(Key)
   {
     case kOk:
-      if(!isempty(Dir))
+      if(state == osUnknown && !isempty(Dir))
       {
         char *buffer = NULL;
         if(0 < asprintf(&buffer, "%s/%s", CurrentDir(), stripspace(Dir))) {
@@ -520,7 +522,7 @@ eOSState cCMDDir::Edit(eKeys Key)
       break;
   }
 
-  return cOsdMenu::ProcessKey(Key);
+  return state;
 }

Alles anzeigen

mini73 · 23. Februar 2014

Ohne das Plugin zu kennen, ist das besser, weil das quasi jedes Menü (auch im vdr) so macht.

Lars

anbr · 23. Februar 2014

Hi,

Ich habe mich auch nur an den Quellen des VDR-Core orientierten, zum historischen Hintergrund der Funktion im Plugin DVDSwitch, kann aber nichts sagen...

Zum Testen, habe ich obigen Patch ins Git-Repository verschoben,

Cu,
Andreas

Jafar · 23. Februar 2014

Ich habe mich nur an der funktionierenden Funktion Image Rename in der menu.c orientiert. Da wird auch erneut ein ProcessKey aufgerufen:

Code

...


eOSState cMainMenu::ProcessKey(eKeys Key)
{
  if(!HasSubMenu())
  {
	switch(State)
	{
  	case mmsImgRename:
    	switch(Key)
    	{
      	case kBack:
        	SetState(mmsNone);
        	DVDSwitchSetup.HideTypeCol = CMDImg->tmpHideTypeCol;
        	DELETENULL(CMDImg);
        	Build(MainMenuOptions.CurrentDir());
        	Display();
        	return osContinue;
        	break;
      	case kOk: {
          	eOSState nState = cOsdMenu::ProcessKey(Key);


          	SetState(mmsNone);
          	DVDSwitchSetup.HideTypeCol = CMDImg->tmpHideTypeCol;
          	if(!isempty(CMDImg->NewFile))


...

Alles anzeigen

Das müßte man dann auch anpassen.

Gruß Jafar

anbr · 23. Februar 2014

Hi,

Zitat von Jafar

Das müßte man dann auch anpassen.

ich stimme Dir zu, aber leider funktioniert es nicht so einfach, im Prinzip müsste das ganze Menühandling um geschrieben werden,
momentan sind die Routinen so unübersichtlich, das ich selber als Maintainer nicht voll durchsteige, was der Autor beabsichtigt hat.

Jafar · 26. Februar 2014

Zitat von anbr

ich stimme Dir zu, aber leider funktioniert es nicht so einfach, im Prinzip müsste das ganze Menühandling um geschrieben werden,
momentan sind die Routinen so unübersichtlich, das ich selber als Maintainer nicht voll durchsteige, was der Autor beabsichtigt hat.

Ich weiß, was Du meint

Gruß Jafar

DVDSwitch: Image von DVD erstellen

Jetzt mitmachen!

Teilen