[Announce] vdr-plugin-tvscraper 1.1.10

kamel5 · Feb 5th 2023

Quote from MarkusE

Letztere Meldung kommt aber nur, wenn bei VDR --log=3 gesetzt ist.

Ja, ist bei mir eingestellt.

Auch "using base directory" gibr es.

Code

Feb 05 09:24:27 home-05.home.de vdr[4275]: [4275] tvscraper: using base directory /etc/vdr/plugins/tvscraper/

Quote from MarkusE

Die Konfiguration ist in der README.md besch

Ja, habe ich gelesen. Da steht aber nichts zu den Verzeichnissen "epg" und "recordings". Oder habe ich da was übersehen.

Ich habe auch bisher in die "override.conf" keine Änderungen eingetragen.

Gestern hatte ich nochmal 2 Abstürze:

Code

#15 0x00007fded7bb53e7 in std::filesystem::__cxx11::directory_iterator::directory_iterator(std::filesystem::__cxx11::path const&, std::filesystem::directory_options, std::error_code*)
    (this=0x7fde7b7fdb30, p=filesystem::path "/etc/vdr/plugins/tvscraper/recordings/" = {...}, options=<optimized out>, ecptr=0x0)
    at /usr/src/debug/gcc-12.2.1-4.fc37.x86_64/obj-x86_64-redhat-linux/x86_64-redhat-linux/libstdc++-v3/include/bits/new_allocator.h:90
        skip_permission_denied = false
        ec = std::error_code = {"generic": ENOENT}
        dir = {<std::filesystem::_Dir_base> = {dirp = 0x0}, path = filesystem::path "", entry = {_M_path = filesystem::path "", _M_type = std::filesystem::file_type::none}}
#16 0x00007fdebe2529d0 in std::filesystem::__cxx11::directory_iterator::directory_iterator(std::filesystem::__cxx11::path const&)
    (this=0x7fde7b7fdb30, __p=filesystem::path "/etc/vdr/plugins/tvscraper/recordings/" = {...}) at /usr/include/c++/12/bits/fs_dir.h:387
#17 0x00007fdebe2348ba in deleteOutdatedRecordingImages(cTVScraperDB const*) (db=0x22a5f30) at /home/vdr/DVB/Test/vdr/PLUGINS/src/tvscraper/movieOrTv.c:996

Das scheint aber jetzt mit Deinem letzten commit auch abgefangen zu werden.

Das ist der Inhalt vom Verzeichnis tvscraper:

Code

[root@home-05 Core]# la /etc//vdr/plugins/tvscraper/
insgesamt 99986
drwxr-xr-x    6 root root        12  5. Feb 10:43 .
drwxr-xr-x   45 root root       280 14. Dez 17:49 ..
drwxr-xr-x    2 root root         2  4. Feb 22:54 epg
-rw-r--r--    1 root root         0  5. Feb 10:43 .EPG_Update
drwxr-xr-x    5 vdr  vdr       4977  5. Feb 10:43 movies
-rw-r--r--    1 root root      1231 17. Okt 2013  override.conf
drwxr-xr-x    2 root root         2  4. Feb 22:54 recordings
-rw-r--r--    1 root root         0  5. Feb 00:24 .recordingsUpdate
drwxr-xr-x 1267 vdr  vdr       1267  5. Feb 10:43 series
-rw-r--r--    1 root root 154861568  5. Feb 10:43 tvscraper2.db
-rw-r--r--    1 vdr  vdr   22479872  1. Mai 2022  tvscraper.db

Display More

Die Frage ist jetzt, warum gibt es überhaupt den Absturz, denn beide Verzeichnisse gibt es ja. Sie sind allerdings leer.

Und die anderen beiden Verzeichnisse "movies" und "series" sind befüllt und werden auch aktualisiert.

Der VDR läuft bei mir unter "root".

Grüße

kamel5

MarkusE · Feb 5th 2023

Im git ist noch ein Update.

Ich rufe directory_iterator() jetzt mit dem ec Parameter auf, damit sollte es keine Dumps mehr geben.

Ich hätte jetzt nicht gedacht, dass ein directory_iterator eine Ausnahme erzeugt, nur weil das Verzeichnis leer ist. Ist aber so. Steht auch in der Doku . Habe ich jetzt gelesen ...

~ Markus

MarkusE · Feb 6th 2023

Im git ist ein Update der README.md, mit der Beschreibung der Verzeichnisse

kamel5 · Feb 8th 2023

Ich muss das nochmal aufwärmen.

Abstürze habe ich keine mehr, aber wahrscheinlich erst einmal nur wegen CheckDirExists().

Eine Sache fällt mir aber noch auf. Dazu habe ich mal in der Datei "movieOrTv.c" folgende Fehlermeldung eingebaut:

Code

void deleteOutdatedRecordingImages(const cTVScraperDB *db) {
// check recording. Delete if this does not exist
if (!CheckDirExists(config.GetBaseDirRecordings().c_str() )) {
    esyslog("tvscraper, ERROR, %s not found", config.GetBaseDirRecordings().c_str());
    return;
}
std::error_code ec;

und in "tools/filesystem.c":

Code

 bool CheckDirExists(const char* dirName) {
     struct statfs statfsbuf;
+    esyslog("tvscraper, CheckDirExists 1");
     if (statfs(dirName,&statfsbuf)==-1) return false;
-    if ((statfsbuf.f_type!=0x01021994) && (statfsbuf.f_type!=0x28cd3d45)) return false;
+    esyslog("tvscraper, CheckDirExists 2");
+    if ((statfsbuf.f_type!=0x01021994) && (statfsbuf.f_type!=0x28cd3d45)) return false; // CRAMFS_MAGIC 0x28cd3d45 TMPFS_MAGIC 0x01021994
+    esyslog("tvscraper, CheckDirExists 3");
     if (access(dirName,R_OK|W_OK)==-1) return false;
+    esyslog("tvscraper, CheckDirExists 4");
     return true;
 }

Display More

Logausgabe dazu:

Code

Feb 08 11:05:23 home-05.home.de vdr[33706]: [33767] tvscraper: scraping Channel Schlager Deluxe S19.2E-133-15-35
Feb 08 11:05:24 home-05.home.de vdr[33706]: [33767] tvscraper: scraping Channel Spiegel Geschichte HD S19.2E-133-16-137
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper: Cleanup Done
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, CheckDirExists 1
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, CheckDirExists 2
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, ERROR, /etc/vdr/plugins/tvscraper/epg/ not found
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, CheckDirExists 1
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, CheckDirExists 2
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper, ERROR, /etc/vdr/plugins/tvscraper/recordings/ not found
Feb 08 11:05:48 home-05.home.de vdr[33706]: [33767] tvscraper: access /etc/vdr/plugins/tvscraper/tvscraper2.db for write
Feb 08 11:05:49 home-05.home.de vdr[33706]: [33767] tvscraper: access to /etc/vdr/plugins/tvscraper/tvscraper2.db finished
Feb 08 11:05:49 home-05.home.de vdr[33706]: [33767] tvscraper: epg scraping done

Display More

Das Verzeichnis ist vorhanden, siehe weiter oben, wird aber trotzdem als Fehler ausgegeben.

Man sieht hier, das er bei dieser Prüfung:

"if ((statfsbuf.f_type!=0x01021994) && (statfsbuf.f_type!=0x28cd3d45)) return false;"

"false" zurück gibt.

OK, ich nutze zwar kein externes EPG, müsste hier aber nicht die Prüfung trotzdem positiv verlaufen?

Warum wird auf diese beiden Dateisysteme geprüft?

Wenn man die ausschließen will, dann wäre die Abfrage allerdings falsch.

Grüße

kamel5

MarkusE · Feb 8th 2023

Hallo kamel5 ,

Vielen Dank für Deine Analyse!

> Abstürze habe ich keine mehr, aber wahrscheinlich erst einmal nur wegen CheckDirExists().

Glaube ich nicht. Ich rufe ja jetzt "std::filesystem::directory_iterator(config.GetBaseDirEpg(), ec)". Und laut Manual (habe ich ja jetzt gelesen ) gibt es damit nur noch bei out of memory Exceptions. Probiere es doch mal aus, und lösche die "CheckDirExists()." Zeilen in "movieOrTv.c".

Aber danke für Deine Analyse, Du hast einen Fehler gefunden.

Ich hatte CheckDirExists() einfach von Lous übernommen, und nicht weiter geprüft. Ich habe es jetzt umbenannt, in CheckDirExistsRam. Da es prüft, ob das Verzeichnis existiert, und in einem RAM Dateisystem ist.

Dann eine neue Methode DirExists geschrieben, die einfach nur prüft, ob das Verzeichnis existiert.

Ein Update ist im git, aber noch relativ ungetestet.

~ Markus

kamel5 · Feb 8th 2023

Quote from MarkusE

Probiere es doch mal aus, und lösche die "CheckDirExists()." Zeilen in "movieOrTv.c".

Ja, das hätte ich machen können, ich wollte aber wissen, warum es bei mir nicht geht.

Mit dem letzten commit gibt es keine Fehlermeldungen mehr.

Grüße

kamel5

[Announce] vdr-plugin-tvscraper 1.1.10

Participate now!

Share

Users Viewing This Thread