[solved] vdr-2.3.1-hide-first-recording-level-v1.patch - Absturz beim sortieren der Aufnahmen

  • Moin,


    bei mir crashed der vdr-2.3.9 mit dem vdr-2.3.1-hide-first-recording-level-v1.patch beim sortieren der Aufnahmen.

    Also ein drücken der Taste '0' im Aufnahmen-Menü führt zum direktem Neustart.

    Ohne den Patch funktioniert es wie es soll.


    MfG


    Edit: Solved -> Patch

    Edited once, last by Saman ().

  • Ich hatte bisher noch keine Zeit, den Patch an vdr 2.3.x anzupassen. Deshalb weiß ich auch noch nicht, wo es haken könnte.


    Lars

  • bisher noch nicht bemwerkt aber kann ich so bestätigen, ist momentan das einzig ungereimte Stelle in meinem 2.3.9


    Christian

    CKone: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G540, 2x 2GB Kingston DDR3, Zotac GT630 1GB, S2-1600, Ocz Agility 3 60GB, LG GH24NS DVD, 15.6" Selbstbau TFT, Harmony 665, CIR Selbstbau - das Ganze im Silverstone SST-SG03B
    CKtwo: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G1610, 2x 2GB Corsair DDR3, Zotac GT630 1GB, TT S2-1600, Ocz Vertex 2 50 GB, 92 Kanal Seduatmo, Harmony 665, atric USB
    CKthree: yavdr-ansible/22.04 LTS/2.6.1/kodi19.3 auf Intel NUC, Celeron J4005, UHD Graphics 600, 4GB Crucial DDR4, Ocz Vertex2 50 GB, Harmony 350

    PowerEdge: Ubuntu Server 16.04 LTS / VDR 2.4.1 auf Dell PowerEdge T20, Xeon E3-1225 v3, 16GB ECC DDR3, 2x Cine S2 V6 mit Duoflex, Samsung 840 EVO 120GB, 3x WD White WD80EZAZ 8TB in SW Raid5



  • Wer möchte, kann das mal an einem gepatchten VDR testen:

    Oder im Patch den Teil für menu.c etwa so anpassen und den Teil für menu.h ganz raus nehmen:

  • Wer möchte, kann das mal an einem gepatchten VDR testen:

    Oder im Patch den Teil für menu.c etwa so anpassen und den Teil für menu.h ganz raus nehmen:

    Patch 1 scheint ok - zumindest kein Absturz mehr.


    Danke

  • Danke für die Rückmeldung.

    Habe es bei mir über den Tag auch so laufen lassen.

    Kein Absturz mehr und das Sortieren funktioniert auch noch.

    Deswegen hier noch der komplette Patch gegen vanilla vdr-2.3.9


    Edit: Basierend auf dem Patch aus dem ppa von seahawk1986 und mit den Änderungen von oben.

    Files

    Edited once, last by Saman ().

  • Danke für den angepassten Patch, ich packe den mal ins PPA für den VDR 2.3.9 (und lasse danach die Plugins neu bauen).

    yaVDR-Dokumentation (Ceterum censeo enchiridia esse lectitanda.)

  • mir war da noch was aufgefallen, damals mit der 2.3.4 - vllt könnt ihr das einmal verifizieren ob das noch so ist und gfs wenn Saman grad son lauf hat auch mal ansehen ;)


    ich hatte das seinerzeit mal aufgeschrieben:


    Quote

    Im Hauptverzeichnis Ordner alphabetisch oben ist => super

    Im Hauptverzeichnis kann ich die Sortierung wie immer mit 0 von Datum auf alphanumerisch umschalten => auch super

    In den Unterverzeichnissen leider nicht umschaltbar und immer nach Datum => wie krieg ich das auf alphanumerisch damit die Episoden in der richtigen Reihenfolge stehen?


    Christian

    CKone: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G540, 2x 2GB Kingston DDR3, Zotac GT630 1GB, S2-1600, Ocz Agility 3 60GB, LG GH24NS DVD, 15.6" Selbstbau TFT, Harmony 665, CIR Selbstbau - das Ganze im Silverstone SST-SG03B
    CKtwo: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G1610, 2x 2GB Corsair DDR3, Zotac GT630 1GB, TT S2-1600, Ocz Vertex 2 50 GB, 92 Kanal Seduatmo, Harmony 665, atric USB
    CKthree: yavdr-ansible/22.04 LTS/2.6.1/kodi19.3 auf Intel NUC, Celeron J4005, UHD Graphics 600, 4GB Crucial DDR4, Ocz Vertex2 50 GB, Harmony 350

    PowerEdge: Ubuntu Server 16.04 LTS / VDR 2.4.1 auf Dell PowerEdge T20, Xeon E3-1225 v3, 16GB ECC DDR3, 2x Cine S2 V6 mit Duoflex, Samsung 840 EVO 120GB, 3x WD White WD80EZAZ 8TB in SW Raid5



  • Hallo Christian,


    das ist leider schon bei der 2.20 so. Würde mich auch über eine Lösung freuen....


    Gruß Micha

  • Hallo Christian,


    das ist leider schon bei der 2.20 so. Würde mich auch über eine Lösung freuen....


    Gruß Micha

    Bei mir nicht. Ich kann auch in Unterverzeichnissen mit der Taste "0" nach Alphabet oder Datum sortieren.

  • und du hast auch das "hide-first-recording-level" feature aktiviert?

    CKone: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G540, 2x 2GB Kingston DDR3, Zotac GT630 1GB, S2-1600, Ocz Agility 3 60GB, LG GH24NS DVD, 15.6" Selbstbau TFT, Harmony 665, CIR Selbstbau - das Ganze im Silverstone SST-SG03B
    CKtwo: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G1610, 2x 2GB Corsair DDR3, Zotac GT630 1GB, TT S2-1600, Ocz Vertex 2 50 GB, 92 Kanal Seduatmo, Harmony 665, atric USB
    CKthree: yavdr-ansible/22.04 LTS/2.6.1/kodi19.3 auf Intel NUC, Celeron J4005, UHD Graphics 600, 4GB Crucial DDR4, Ocz Vertex2 50 GB, Harmony 350

    PowerEdge: Ubuntu Server 16.04 LTS / VDR 2.4.1 auf Dell PowerEdge T20, Xeon E3-1225 v3, 16GB ECC DDR3, 2x Cine S2 V6 mit Duoflex, Samsung 840 EVO 120GB, 3x WD White WD80EZAZ 8TB in SW Raid5



  • Die Ursache für das Problem konnte ich ausmachen:

    Der VDR sucht im 'local' Verzeichnis nach der '.sort' Datei, findet sie aber nicht und kann auch keine anlegen, da das zu sortierende Verzeichnis dort nicht existiert.


    Als Test oder workaround könnt ihr einfach ein leeres Verzeichnis (mit dem selben Namen wie das zu sortierende) im 'local' Verzeichnis erstellen.

    Darin legt der VDR dann beim drücken auf '0' eine eigene '.sort' Datei ab. (Bedenkt aber, das leere Verzeichnisse beim Aufräumen entfernt werden!)


    Jetzt stellt sich die Frage, ob alle Clients die selbe Sortierung nutzen wollen/sollen oder nicht.

    Je nachdem, wäre das dann ein Bug oder ein Feature ;)

  • Hi,

    Wäre das nicht noch ein Bug der 2.3.9,der abgestellt werden sollte?

    MfG Stefan

    Test-VDR1: HP rp5700 Fertigsystem, Core2Duo E6400, 2GB RAM, FF-SD C-2300, nvidia Slim-GT218 x1 | easyVDR 2.0 64Bit
    VDR3: in Rente

    VDR4: MSI G31M2 v2, Digitainer2-Geh., t6963c 6" gLCD, E5200, 2GB, 3TB WD Red, GT730, 2x TT S2-3200; easyVDR 3.5 64bit
    VDR5: Gigabyte
    GA-G31M-S2L, Intel E2140, Zotac GT730 passiv, Digitainer2-Geh., t6963c 6 " gLCD, 2 TB WD Red, 2x TT S2-3200 (an 1 Kabel) easyVDR 3.5 64bit
    VDR6:
    Intel E5200, GT630 passiv, F1 750 GB, t6963c gLCD, 2x TT S2-3200 | easyVDR 3.5 64bit
    VDR-User #1068
    www.easy-vdr.de

  • ja ich glaube auch, der workaround taugt maximal für eine Demonstration das es so funktioniert.


    mE sollte das an der Wurzel gefixt werden.


    Christian

    CKone: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G540, 2x 2GB Kingston DDR3, Zotac GT630 1GB, S2-1600, Ocz Agility 3 60GB, LG GH24NS DVD, 15.6" Selbstbau TFT, Harmony 665, CIR Selbstbau - das Ganze im Silverstone SST-SG03B
    CKtwo: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G1610, 2x 2GB Corsair DDR3, Zotac GT630 1GB, TT S2-1600, Ocz Vertex 2 50 GB, 92 Kanal Seduatmo, Harmony 665, atric USB
    CKthree: yavdr-ansible/22.04 LTS/2.6.1/kodi19.3 auf Intel NUC, Celeron J4005, UHD Graphics 600, 4GB Crucial DDR4, Ocz Vertex2 50 GB, Harmony 350

    PowerEdge: Ubuntu Server 16.04 LTS / VDR 2.4.1 auf Dell PowerEdge T20, Xeon E3-1225 v3, 16GB ECC DDR3, 2x Cine S2 V6 mit Duoflex, Samsung 840 EVO 120GB, 3x WD White WD80EZAZ 8TB in SW Raid5



  • Da das Sortieren funktioniert, wenn der Patch deaktiviert ist oder nicht angewandt wurde, ist das IHMO kein Bug - eher ein 'Feature Request'.

    Für den Patch könnte man natürlich über sowas wie '.sort.local' oder '.sort.$hostname' nachdenken.

  • Moin, ich habe mir mal ein paar Debug Ausgaben in menu.c eingebaut.


    Das Problem liegt wohl in cMenuRecordings::DirectoryName()

    Code
         if (cVideoDirectory::HideFirstRecordingLevel()) {
            LOCK_RECORDINGS_READ;
            cRecordings::cFolderInfos::cFolderInfo* info = Recordings->GetFolderInfo(base);
            if (info) {
               if (info->FirstFolderNames.Size() > 0)
                  d = AddDirectory(d, info->FirstFolderNames.At(0));
               delete info;
               }
            }

    Beim öffnen des Aufnahmen-Menüs gibt die Funktion '/srv/vdr/video' zurück.

    Öffnet man dann ein Verzeichnis, das in 'local' liegt, kommt zB '/srv/vdr/video/local//TEST'.

    Öffnet man stattdessen ein externes Verzeichnis, kommt zB '/srv/vdr/video/local//Spielfilme'.


    Merkwürdig an dieser Stelle finde ich, das 'FirstFolderNames' nur 'local/' enthält.

    Sollte das nicht auch die anderen Verzeichnisse aus '/srv/vdr/video' beinhalten?

    Dann könnte in einem Loop geprüft werden, ob/bis das Verzeichnis existiert...

    Ich denke, das wäre die allgemein gültigste Variante.


    Wer die Aufnahmen ausschließlich auf einem Server liegen hat, kann es auch hart kodieren:

    Code
    if (cVideoDirectory::HideFirstRecordingLevel()) {
       d = AddDirectory(d, "vdrserver");
    }
  • So, ich glaube, ich hab es jetzt und das von oben ist zum Teil Blödsinn :)

    Es fehlte einfach das  == 0 Mit dieser Änderung kann ich jetzt alle Verzeichnisse sortieren.

  • Prima, damit klappt die Sortierung der Aufnahmen in Unterverzeichnissen wieder :)

    yaVDR-Dokumentation (Ceterum censeo enchiridia esse lectitanda.)

  • Hallo Christian,


    das ist leider schon bei der 2.20 so. Würde mich auch über eine Lösung freuen....


    Gruß Micha


    Ich habe eben noch mal im vdr-2.2.0-hide-first-recording-level-v5.patch geschaut, da fehlt das == 0 auch.

  • dann kann die Version 2.4.0 ja zum WE kommen! :mua


    Vielen Dank für den Fix

    CKone: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G540, 2x 2GB Kingston DDR3, Zotac GT630 1GB, S2-1600, Ocz Agility 3 60GB, LG GH24NS DVD, 15.6" Selbstbau TFT, Harmony 665, CIR Selbstbau - das Ganze im Silverstone SST-SG03B
    CKtwo: yavdr-ansible/18.04 LTS/2.4.1/kodi18 auf Intel DH67BL, Intel Celeron G1610, 2x 2GB Corsair DDR3, Zotac GT630 1GB, TT S2-1600, Ocz Vertex 2 50 GB, 92 Kanal Seduatmo, Harmony 665, atric USB
    CKthree: yavdr-ansible/22.04 LTS/2.6.1/kodi19.3 auf Intel NUC, Celeron J4005, UHD Graphics 600, 4GB Crucial DDR4, Ocz Vertex2 50 GB, Harmony 350

    PowerEdge: Ubuntu Server 16.04 LTS / VDR 2.4.1 auf Dell PowerEdge T20, Xeon E3-1225 v3, 16GB ECC DDR3, 2x Cine S2 V6 mit Duoflex, Samsung 840 EVO 120GB, 3x WD White WD80EZAZ 8TB in SW Raid5



Participate now!

Don’t have an account yet? Register yourself now and be a part of our community!