audiorecorder version 0.1.0-pre6

machtnix · 15. Oktober 2007

Das hat mir jetzt doch keine Ruhe gelassen :

Hab's jetzt mit ffmpeg-cvs und libmp3lame am laufen !!

Nachdem ich alles nochmal neu zusammengepatcht habe und die Namensgebung auf "extern" eingestellt habe, klappt es jetzt auch mit mp3s einwandfrei.

Leider ist mein Source mit 66k zu groß um ihn hier upzuloaden.

prudentis · 15. Oktober 2007

Hört sich doch gut qn,

Beschreib einfach kurz, was du wo gemacht hast, dann krieg ich das schon hin!

lg prudentis

machtnix · 15. Oktober 2007

Habe mal ein diff zur original 0.1.0pre6 erstellt
1. Config Ordner - /etc/vdr/plugins/audiorecorder erstellen
2. Plugin patchen und übersetzen
3. den Inhalt des contrib Ordners in den Config Ordner kopieren
4. im Pluginsetup Audiokodierung auf libmp3lame und Dateimuster auf External stellen
5. Pluginstartparameter im setup einstellen (-p /audiopfad -r /etc/vdr/plugins/audiorecorder/set_path)

viel Glück

machtnix

I30R6 · 15. Oktober 2007

Nabend,

hmm kann es sein , dass der Patch nur für vdr 1.4.x ist ?

bei mir kommt unter vdr 1.5.9

Code

vdr:/usr/local/src/vdr-1.5.9/PLUGINS/src/audiorecorder# make all
make: *** Keine Regel vorhanden, um das Target »xml-cache.h«,
  benötigt von »audiorecorder.o«, zu erstellen.  Schluss.
vdr:/usr/local/src/vdr-1.5.9/PLUGINS/src/audiorecorder# mc

als würde was fehlen.

I30R6

machtnix · 15. Oktober 2007

Ja, ist nur für vdr 1.4.x

aber eine Datei hatte trotzdem gefehlt
Der Anhang ist jetzt korrigiert
Hoffe jetzt ist alles ok

prudentis · 15. Oktober 2007

Hallo machtnix,

krieg einen Fehler beim Patchen:

Code

zcat libmp3lame-audiorecorder.diff.gz | patch
patching file audioreceiver.c
patching file audiorecorder.c
patching file audiorecorder.h
patching file audiorecorder_i18n.c
patching file audiorecorder_i18n.h
patching file browse.c
patching file cache.c
patching file cache.h
patching file audiorecorder.conf
patching file postproc.pl
patching file postproc.sh
patching file set_path
patching file dispatcher.c
patching file dispatcher.h
patching file HISTORY.magicamun
patching file mainmenu.c
patching file mainmenu.h
patching file Makefile
patching file postdata.c
patching file postdata.h
patching file postproc.c
patching file postproc.h
patching file README
patching file README.magicamun
patching file setup.c
patching file setup.h
can't find file to patch at input line 2808
Perhaps you should have used the -p or --strip option?
The text leading up to this was:
--------------------------
|diff -Nur /audiorecorder-0.1.0-pre6/tinyxml/tinyxml.h /usr/local/src/VDR/PLUGINS/src/audiorecorder-0.1.0-pre6/tinyxml/tinyxml.h
|--- /audiorecorder-0.1.0-pre6/tinyxml/tinyxml.h        2006-09-05 17:45:16.000000000 +0200
|+++ /usr/local/src/VDR/PLUGINS/src/audiorecorder-0.1.0-pre6/tinyxml/tinyxml.h  2007-10-15 18:32:29.000000000 +0200
--------------------------
File to patch:

Alles anzeigen

lg prudentis

machtnix · 16. Oktober 2007

Bei File to patch: den Pfad zur Datei angeben

/usr/local/src/VDR/PLUGINS/src/audiorecorder-0.1.0-pre6/tinyxml/tinyxml.h

Keine Ahnung ob man bei der diff-erstellung das anders machen kann.

prudentis · 16. Oktober 2007

Hallo machtnix,

habs jetzt mal folgendermaßen gemacht,

Tinyxml diff drüberlaufen lassen, dann dein diff,

danach hat er sauber kompiliert, habs im easyvdr portal bereit gestellt, und warte mal ab was die User sagen. Konnte leider noch nicht selber testen!

http://www.easy-vdr.de/forum/i…ic=2464.msg19773#msg19773

Falls das nach meiner Methode nicht geht, probier ich es mit deiner nochmal.

Auf jeden Fall erstmal FETTES DANKE für deine Hilfe

lg prudentis

machtnix · 16. Oktober 2007

@ ark

Da ich zum Aufnehmen das libmp3lame benutze werden vom Audiorecorder die Dateien mit der Endung .libmp3lame versehen.

In Deiner Erweiterung mit den Scripten kann man ja eine externe Nachbearbeitung der Dateinamen machen lassen.
Das funktioniert auch soweit, dass ich die Dateiendung auf .mp3 ändern lasse.

echo -n "$3_-_$4.mp3" |tr '[A-Z]' '[a-z]' |sed 's/ /_/g'

Kann ich diese Parameter auch dahingehend ändern, dass Der Interpret als Ordner und der Titel darin gespeichert wird ?

Gruß
machtnix

ARK · 16. Oktober 2007

Hi machtnix,

die Initialisierungsroutinen vom audiorecorder sind mir zu verworren programmiert, daher weiß ich nicht, wie man die Dateiendung sauber ausprogrammieren könnte ohne größere Umbauten vorzunehmen.

Eine Verzeichnisstruktur lässt sich doch einfach realisieren, Du musst nur statt dem Trenner den Verzeichnisstrich einbauen:

Code

echo -n "$3/$4.mp3" |tr '[A-Z]' '[a-z]' |sed 's/ /_/g'

Ich meine, der audiorecorder legt das Verzeichnis auch an, ich kann das hier jedoch gerade nicht aus probieren Ansonsten müsste man noch Klimmzüge machen.

Gruß,
ARK

machtnix · 16. Oktober 2007

ja, so funktionierts

Das war ja mal erfreulich einfach

Thank's

magicamun · 17. Oktober 2007

ich hab irgendwie verpasst, wieso hier so kompliziert dedacht/gearbeitet wird :

"Normalerweise" ermittelt audiorecorder anhand von ladbaren libs, welche Codecs vorhanden sind, und stellt die default - Endungen im Setup zur Verfügung (mp2/mp3) - geht das schief oder wie ?

Der audiorecorder legt dann die Dateien so ab, wie eingestellt, oder aber im Externen Script definiert.

Ich komm aktuell nicht dazu in den Quellen nachzuschauen und hab auch kein EasyVdr - aber so müsste es laufen.

ARK · 17. Oktober 2007

Hallo magicamun,

wie schon hier beschrieben, wird im aktuellen Code mittels avcodec_find_encoder_by_name("mp3") der mp3-Encoder gesucht, leider heißt der aber neuerdings nicht mehr mp3 sonder mp3lame, also funktioniert beispielsweise avcodec_find_encoder_by_name("libmp3lame"). Leider ist jedoch in der Konfiguration vom audiorecoder keine Trennung zwischen encoder-Name und Codierung, so dass beim Patch von machtnix auch die Dateiendung jetzt mp3lame heißt - unschön!

IMHO wäre der richtige Weg im audiorecorder die Methode avcodec_find_encoder(CODEC_ID_MP3) aufzurufen, dazu müsste man jedoch den Initialisierungsteil aufbohren, usw. Soweit meine bisherige Analyse. Ich habe noch den alten lame, mein Leidensdruck ist da bisher nicht so groß

Gruß,
ARK

I30R6 · 17. Oktober 2007

Grüß Gott,

so beim EasyVDR Forum sind wir das Ganze anders rum gegangen .

Altes FFmpeg 0.4.9 gepatcht damit es den neuen 4.x.x gcc kompiliert wird ein paar Dateien umkopiert und schon kann die HDD mit mp3 gefüllt werden .

I30R6

skiller2k1 · 17. Oktober 2007

Zitat

Original von ark
IMHO wäre der richtige Weg im audiorecorder die Methode avcodec_find_encoder(CODEC_ID_MP3) aufzurufen, dazu müsste man jedoch den Initialisierungsteil aufbohren, usw. Soweit meine bisherige Analyse.

Ich verstehe auch nicht wirklich, warum man sich so einen großen Streß macht um FFMPEG zu downgraden?

Wie ich hier:

audiorecorder version 0.1.0-pre6

bereits geschrieben habe, reicht es vollkommen aus diese zwei Zeilen zu patchen und es läuft.

Hier nochmal im Detail:

Diff

diff -urNad vdr-plugin-audiorecorder-0.1.0-pre9~/audiorecorder.c vdr-plugin-audiorecorder-0.1.0-pre9/audiorecorder.c
--- vdr-plugin-audiorecorder-0.1.0-pre9~/audiorecorder.c	2007-07-05 13:31:17.000000000 +0200
+++ vdr-plugin-audiorecorder-0.1.0-pre9/audiorecorder.c	2007-07-05 13:31:30.000000000 +0200
@@ -172,7 +172,7 @@
         avcodec_init();
         avcodec_register_all();

-        probe_audio_codecs();
+/*        probe_audio_codecs(); */

         Cache.load();        

diff -urNad vdr-plugin-audiorecorder-0.1.0-pre9~/convert.c vdr-plugin-audiorecorder-0.1.0-pre9/convert.c
--- vdr-plugin-audiorecorder-0.1.0-pre9~/convert.c	2007-07-05 13:26:12.000000000 +0200
+++ vdr-plugin-audiorecorder-0.1.0-pre9/convert.c	2007-07-05 13:31:17.000000000 +0200
@@ -70,7 +70,7 @@
 void cConvert::init_encoder(const char *codec, int bit_rate, int sample_rate,
         int channels)
 {
-        encoder_codec = avcodec_find_encoder_by_name(codec);
+        encoder_codec = avcodec_find_encoder(CODEC_ID_MP3);
         if (! encoder_codec) {
                 dsyslog("[audiorecorder]: codec %s is not supported (%s, "
                         "%s())", codec, __FILE__,  __func__);

Alles anzeigen

So läuft es bei mir mit der aktuellsten FFMPEG-SVN unter Debian Sid/Etch - ohne Probleme!

Mit Sicherheit ist das die temporär eleganteste Lösung. Spätestens wenn das Plugin nur noch FFMPEG-Versionen unterstützt, die den neuen Codec-String benutzen müsste man sich ja schon wieder den Streß machen FFMPEG neuzubauen..

Grüße
Michi

I30R6 · 17. Oktober 2007

Mahlzeit , Michi

ich habe deinen Patch auch schon mehrmals probiert .

Das Ergebnis : Dateien haben die Endung .mp3
aber leider sind die Dateien viel zu klein und lassen sich nicht abspielen.

//Edit :

Ich hätte nen kleinen Wunsch .. kann man unter Dateimuster noch sowas einbringen : Kanal/Künstler-Titel ?? der Übersicht halber

I30R6

machtnix · 17. Oktober 2007

@ skiller2k1
Das kann ich ebenfalls bestätigen.

Das konvertieren mit der aktuellen CVS Version von ffmpeg bringt bei mir auch nur 2kb große Dateien.

I30R6 · 17. Oktober 2007

:mahlzeit,

mir ist noch ein Problem aufgefallen , wenn ich

KANAL/ARTIST/TITEL wähle dann erstellt er die mp3s aber nur so ARTIST - TITEL

wie schon erwähnt mir wäre

KANAL/ARTIST - TITEL wäre optimal

//Edit

Code

switch(_file_pattern) {
            case 0 :                    // Artist/Title
                tmp << artist << "/" << title << "." << codec;
                break;
            case 1 :                    // Artist - Title
        tmp << artist << "-" << title << "." << codec;
                break;
            case 2 :                    // Station/Artist/Title
                tmp << artist << "/" << title << "." << codec;
                break;
            case 3 :                    // Station - Artist - Title
                tmp << artist << "-" << title << "." << codec;
                break;
            case 4 :                    // External
                tmp << path_external();
                break;

Alles anzeigen

Also ganz richtig kann das nicht sein.

//Edit2 :

die Zeile müsste so heißen

Code

case 2 :                    // Station/Artist/Title
                tmp << channel << "/" << artist << "/" << title << "." << codec;

leider wird nur der Ordner "channel" erstellt nicht aber die mp3 dort rein geschrieben

//Edit3:
habe es jetzt für meine Ansprüche angepasst :

Code

case 2 :                    // Station/Artist/Title
                tmp << channel << "/" << artist << "-" << title << "." << codec;

sicher muss das Ganze noch woanders geändert werden , Aber mir reichts

so wird ein Verzeichnis für jeden Sender angelegt und dort kommen die MP3s direkt rein.

I30R6

machtnix · 17. Oktober 2007

Hi I30R6,

Die Namensgebung funktioniert gut mit den Skripten von ark.
Im Plugin auf Externe Namensgebung umstellen. Den Pfad zu den Scripten im Setup des Plugins einstellen mit -p xxx/xxx/set_path

Die letzte Zeile in der set_path nach eigenem Wunsch anpassen
z.B. echo -n "$5/$3" - "$4.$7" sollte das so machen wie Du es willst.

Gruß
machtnix

I30R6 · 17. Oktober 2007

Zitat

Original von machtnix
Hi I30R6,

Die Namensgebung funktioniert gut mit den Skripten von ark.
Im Plugin auf Externe Namensgebung umstellen. Den Pfad zu den Scripten im Setup des Plugins einstellen mit -p xxx/xxx/set_path

Die letzte Zeile in der set_path nach eigenem Wunsch anpassen
z.B. echo -n "$5/$3" - "$4.$7" sollte das so machen wie Du es willst.

Gruß
machtnix

Alles anzeigen

Hallo auch , machtnix

habe bereits so wie ich es gern hätte (ohne das Skript) so erstmal die HDD füllen .

I30R6

audiorecorder version 0.1.0-pre6

Jetzt mitmachen!

Teilen