Script in reccmds.conf funzt nicht mit allen Aufnahmen

maverick-me · 7. Juni 2005

Hallo,

wenn ich zum Bsp. einen Eintrag in meiner reccmds.conf habe

Code

MPEG erstellen : vdrsync.sh

dann wird beim Aufrufen des Eintrags, zum Ausführen einfach der
Name der aktuell ausgewählten Aufnahme an den Befehl rangehangen.
Funzt sowei auch, aber wenn nun die Aufnahme ein "&"-Zeichen z.B.
"/video/Tiger_&_Dragon/2005-xxxxxxxxxxxxx"
enthält, läuft gar nix mehr .
Es werden dann 2 Befehle ausgefürt
1. "vdrsync.sh /video/Tiger_"
klar nach dem "&" wird abgeschnitten und seperat ausgeführt,
so kann das aber nicht laufen da die Pfadangabe nicht komplet ist
2. "_Dragon/2005-xxxxxxxxxxxxx" aus!
der Befehl ist dann sowieso Unsinn
Anstatt beise zusammen
"vdrsync.sh /video/Tiger_&_Dragon/2005-xxxxxxxxxxxxx"

gibts in der reccmds.conf irgendeine Möglichkeit den Aufnahmenamen als
Placeholder mit einzutragen, damit ich darum ein paar ' machen kann?
Oder hat jemand ne andere Lösung für das Problem?

Mag1c · 7. Juni 2005

Da wirst du wohl um das Patchen des VDR nicht drumherumkommen
So ganz aus der kalten geschossen:

in recordings.c:

Code

void cRecordingUserCommand::InvokeCommand(const char *State, const char *RecordingFileName)
{
  if (command) {
     char *cmd;
     asprintf(&cmd, "%s %s \"%s\"", command, State, *strescape(RecordingFileName, "\"$"));
     isyslog("executing '%s'", cmd);
     SystemExec(cmd);
     free(cmd);
     }
}

baue mal hinter RecordingFileName noch ein & mit ein:

Code

void cRecordingUserCommand::InvokeCommand(const char *State, const char *RecordingFileName)
{
  if (command) {
     char *cmd;
     asprintf(&cmd, "%s %s \"%s\"", command, State, *strescape(RecordingFileName, "\"$&"));
     isyslog("executing '%s'", cmd);
     SystemExec(cmd);
     free(cmd);
     }
}

Absolut ungetestet und ohne Gewähr.

Gruß
Mag1c

maverick-me · 7. Juni 2005

Hi, so habs selber gefunden!
Der VDR übergibt den Pfad schon in '-Zeichen mein Script hat das dann auch richtig an vdrsync weitergegeben!

Ich habe aber ausserdem noch im Script aus dem Pfad den Aufnahmenamen
als basename an vdrsync weitergegeben, gedemuxt wurde alles super nur bei
der Übergabe an den Multiplexer wurden die einzelnen Streams nicht in
"-Zeichen an den Multiplexer Übergeben somit hat der dan den Mist gebaut
hab einfach vdrsync an ein paar stellen geändert und nun läufts.

Mag1c
Trotzdem Danke für die Antwort.

marcmerz · 7. Juni 2005

Hi maverick-me,

könntest Du ein Diff erstellen?

Gruß,
Marcus

maverick-me · 7. Juni 2005

Hier das diff

Diff

--- vdrsync.pl  Tue Mar 22 20:45:40 2005
+++ vdrsync.pl-patched  Tue Jun  7 17:11:33 2005
@@ -576,12 +576,12 @@


     my $mplex_command;


-    $mplex_command = "nice -n 19 $progs{tcmplex_panteltje} -i $filestomux[0] -0 $filestomux[1]";
+    $mplex_command = "nice -n 19 $progs{tcmplex_panteltje} -i \"$filestomux[0]\" -0 \"$filestomux[1]\"";
     dprint("Mplexing using tcmplex-panteltje\n");
     my $counter = 2;
     while (scalar(@filestomux) > $counter) {
         dprint("adding to $counter $filestomux[$counter]\n");
-        $mplex_command .= " -" . ($counter-1) . " $filestomux[$counter]";
+        $mplex_command .= " -" . ($counter-1) . " \"$filestomux[$counter]\"";
         $counter++;
     }
     $remuxfilename .= "$basename.mpg";
@@ -611,9 +611,9 @@


     my $mplex_command;


-    $mplex_command = "nice -n 19 $progs{tcmplex} -i $filestomux[0] -p $filestomux[1]";
+    $mplex_command = "nice -n 19 $progs{tcmplex} -i \"$filestomux[0]\" -p \"$filestomux[1]\"";
     if (scalar(@filestomux) > 2) {
-        $mplex_command .= " -s $filestomux[2]";
+        $mplex_command .= " -s \"$filestomux[2]\"";
     }
     $remuxfilename .= "$basename.mpg";


@@ -658,11 +658,11 @@


     }
     $mplex_command .= " -o $target ";
-    $mplex_command .= " $filestomux[0]  $filestomux[1]";
+    $mplex_command .= " \"$filestomux[0]\"  \"$filestomux[1]\"";
     my $counter = 2;
     while (scalar(@filestomux) > $counter) {
         dprint("adding to $counter $filestomux[$counter]\n");
-        $mplex_command .= " $filestomux[$counter]";
+        $mplex_command .= " \"$filestomux[$counter]\"";
         $counter++;
     }

Alles anzeigen

marcmerz · 8. Juni 2005

Klasse. Vielen Dank!

Gruß,
Marcus

Script in reccmds.conf funzt nicht mit allen Aufnahmen

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema