Fehler in recording-hooks

TCr82 · 3. Juni 2012

Hi,

bin gerade durch zufall auf ein Problem gestoßen:

Sobald eine Aufnahme eine Klammer im Namen trägt gibt es Fehler in den recording-hooks

sh: Zeile 23: Syntaxfehler beim unerwarteten Wort `('
sh: Zeile 23: `/usr/share/vdr/recording-hooks/R60copy_epgimage after /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec &> /dev/null'

das ist bei allen Skripten zu finden: R50correct_videodirs R60copy_epgimage R90.custom

Lässt sich sicher irgendwie korrigieren... evtl. den Pfad einfach mit Gänzefüßlein umschlingen?

Gruß,
Thomas

TCr82 · 3. Juni 2012

ok.. auf jeden fall passt da was nicht

bin nebenbei so an die Fehler ran gekommen:

Code

sudo su -c mail vdr

Gruß

TCr82 · 3. Juni 2012

ich habe die ja nicht da rein gepackt... was bitte soll ich da ändern?

Ich drücke auf Timer setzten und alles andere passierte automatisch...

TCr82 · 3. Juni 2012

also ich habe nochmal einen test gemacht und das skript nach temp kopiert die aktionen unter den drei genannten Zeilen entfernt und so aufgerufen:

Code

thomas@vdr:/tmp$ ./R60copy_epgimage after /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec
-bash: Syntaxfehler beim unerwarteten Wort `('
thomas@vdr:/tmp$ ./R60copy_epgimage after "/srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec"
+ case $1 in
+ echo 'Copy EPG-Image to /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec'
Copy EPG-Image to /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec
++ grep '^E ' '/srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec/info'
+ ELINE='E 60798 1336651200 2700 4E 9'
+ ELINE='60798 1336651200 2700 49'
+ EVENTID=60798
thomas@vdr:/tmp$ ./R60copy_epgimage after /srv/vdr/video.00/Die_Odyssee_des_Menschen_\(2~3\)/2012-05-10.13.58.210-0.rec/
+ case $1 in
+ echo 'Copy EPG-Image to /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec/'
Copy EPG-Image to /srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec/
++ grep '^E ' '/srv/vdr/video.00/Die_Odyssee_des_Menschen_(2~3)/2012-05-10.13.58.210-0.rec//info'
+ ELINE='E 60798 1336651200 2700 4E 9'
+ ELINE='60798 1336651200 2700 49'
+ EVENTID=60798

Alles anzeigen

wie man sieht muss es an dem Namen mit der Klammer liegen... es ist dem ganz egal was für ein Inhalt das ist.. wieso er auch immer die Zeile 23 genannt hatte.

TCr82 · 3. Juni 2012

ich denke, der fehler ist in der Datei /usr/lib/vdr/vdr-recordingaction

Dort sollte der Pfad gequotet werden.

also so:

Diff

--- /usr/lib/vdr/vdr-recordingaction	2012-05-08 18:34:53.654364752 +0200
+++ /tmp/vdr-recordingaction	2012-06-03 22:12:52.013991621 +0200
@@ -46,6 +46,6 @@
 
 for recordinghook in $recordinghooks; do
     logger -t recordingaction "executing $recordinghook $action"
-    echo "$recordinghook $* &> /dev/null" | at now
+    echo "$recordinghook $1 \"$2\" &> /dev/null" | at now
 done

TCr82 · 3. Juni 2012

fnu

wie soll ich das jetzt deuten, dass du all deine antworten gelöscht hast?

wird das Problem nun totgeschwiegen? Sollen Benutzer besser keine Probleme melden?

steffen_b · 4. Juni 2012

Nicht überbewerten

Sind das Maximum an Argumenten 2 ?

TheChief · 4. Juni 2012

Seit 1.7.28 gibt es neu für den "edited" hook ein optionales Argument mehr, der den Pfad der Source-Aufnahme mit übergibt. Nützlich wenn man EPG Bilder beim Schneiden einer Aufnahme mit kopieren will.

Die Diskussion dazu gabs hier: Aufnahme-(EPG)-Images bei/nach Schnitt

CKone · 4. Juni 2012

danke für den Hinweis, hatte ich nicht gesehen

hotzenplotz5 · 4. Juni 2012

ich habs mal nach da:
https://bugs.yavdr.com/issues/779

wenn mir jemand sein ok gibt für den patch bzw. steffen mir das bestätigt.
dann geht der atch von aLiEnTxC auch gleich nach github.
aber wie gesagt, mir wäre recht wenn steffen was dazu sagt (skriptmeister ...)

TheChief · 4. Juni 2012

Wie gesagt, ich denke ab 1.7.28 müsste man noch prüfen, ob es noch ein 3. Argument gibt. Wie man das macht, würde mich auch interessieren.

Vielleicht so?

Code

anzahl=$#


if [ "$anzahl" -eq "3" ]; then
    echo "$recordinghook $1 \"$2\" \"$3\" &> /dev/null" | at now
else
    echo "$recordinghook $1 \"$2\" &> /dev/null" | at now
fi

Keine_Ahnung · 4. Juni 2012

Code

echo "$recordinghook $1 \"$2\" ${3:+\"$3\"} &> /dev/null" | at now

aber eigentlich sollte nix dagegen sprechen (auch wenn $3 leer ist)

Code

echo "$recordinghook $1 \"$2\" \"$3\" &> /dev/null" | at now

cu

TheChief · 4. Juni 2012

Warum einfach, wenns auch kompliziert geht. Danke, werd mal meine Skripte anpassen.

Keine_Ahnung · 4. Juni 2012

Ich denke um es wirklich richtig zu machen sollte man bash nehmen.

Code

echo "$recordinghook $1 \"$(printf "%q" "$2")\" \"$(printf "%q" "$3")\" &> /dev/null" | at now

Code

root@dirk-vdr:~# echo $(printf "%q" "'foo * ? / : ()")
\'foo\ \*\ \?\ /\ :\ \(\)
root@dirk-vdr:~#

Dann hat man alle Fälle erschlagen weil alle speziellen Zeichen sauber gequotet werden.

dash (/bin/sh per default unter Debian/Ubuntu) kann so was anscheinend nicht.

cu

gda · 4. Juni 2012

Zitat von Keine_Ahnung

dash (/bin/sh per default unter Debian/Ubuntu) kann so was anscheinend nicht.

ACK, deshalb ist die BASH die default sh bei yaVDR.

Gerand

hotzenplotz5 · 4. Juni 2012

ist ja ne alpha, da alle testen dürfen hab ich die variante von keine_ahnung gepusht
https://github.com/yavdr/yavdr…661138f636225238f267f2c00

besten dank dafür !

merci auch an aLiEnTxC fürs melden !

hueb_s · 4. Juni 2012

Das Problem hab ich auch schon seit 'ner Weile - dachte immer, ich bin der einzige...
Immer, wenn Umlaute/Sonderzeichen/Klammern etc. im Dateinamen der Recordings drin sind.

TCr82 · 4. Juni 2012

danke fürs korrigieren...

Könnte man e-Tobi evtl. zurückmelden, da das soweit ich weiss aus seinen Debian Paketen resultierte...

Ich schicke ihm mal ne PN...

Keine_Ahnung · 5. Juni 2012

Ich fürchte die Variante mit printf funktioniert doch nicht.

Das mit \"$2\" schient aber gut zu reichen.

cu

steffen_b · 5. Juni 2012

Das Ausführen der Hooks über at ist eine Erweiterung von mir. Das Problem ist, das die Skripte nur kurz laufen dürfen, da sonst der VDR blockiert.

Vielen Dank für den Input - ich schau nachher nochmal drauf

Fehler in recording-hooks

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema