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

Saman · April 8, 2018 at 7:10 AM

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

mini73 · April 8, 2018 at 10:17 AM

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

CKone · April 8, 2018 at 10:48 AM

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

Christian

Saman · April 8, 2018 at 2:32 PM

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

Diff

pi@raspberrypi:~/Entwicklung $ diff -ruN vdr-2.3.9++/menu.c vdr-2.3.9/menu.c
--- vdr-2.3.9++/menu.c    2018-03-26 20:16:28.134379878 +0200
+++ vdr-2.3.9/menu.c    2018-04-08 13:03:20.346681250 +0200
@@ -3009,7 +3009,8 @@
         CurrentRecording = *fileName ? *fileName : cReplayControl::LastReplayed();
      int current = Current();
      Clear();
-     GetRecordingsSortMode(DirectoryName(Recordings));
+//     GetRecordingsSortMode(DirectoryName(Recordings));
+     GetRecordingsSortMode(DirectoryName());
      Recordings->Sort();
      cMenuRecordingItem *CurrentItem = NULL;
      cMenuRecordingItem *LastItem = NULL;
@@ -3066,11 +3067,13 @@
   fileName = FileName;
 }
 
-cString cMenuRecordings::DirectoryName(const cRecordings *Recordings)
+//cString cMenuRecordings::DirectoryName(const cRecordings *Recordings)
+cString cMenuRecordings::DirectoryName()
 {
   cString d(cVideoDirectory::Name());
   if (base) {
      if (cVideoDirectory::HideFirstRecordingLevel()) {
+        LOCK_RECORDINGS_READ;
         cRecordings::cFolderInfos::cFolderInfo* info = Recordings->GetFolderInfo(base);
         if (info) {
            if (info->FirstFolderNames.Size() > 0)
@@ -3263,8 +3266,9 @@
      return osContinue;
   if (const cMenuRecordingItem *ri = (cMenuRecordingItem *)Get(Current()))
      SetRecording(ri->Recording()->FileName()); // makes sure the Recordings menu will reposition to the current recording
-  cRecordings *Recordings = cRecordings::GetRecordingsWrite(recordingsStateKey); // write access is necessary for sorting!
-  IncRecordingsSortMode(DirectoryName(Recordings));
+//  cRecordings *Recordings = cRecordings::GetRecordingsWrite(recordingsStateKey); // write access is necessary for sorting!
+//  IncRecordingsSortMode(DirectoryName(Recordings));
+  IncRecordingsSortMode(DirectoryName());
   recordingsStateKey.Reset();
   Set(true);
   return osContinue;
pi@raspberrypi:~/Entwicklung $ diff -ruN vdr-2.3.9++/menu.h vdr-2.3.9/menu.h
--- vdr-2.3.9++/menu.h    2018-03-26 17:20:13.376544110 +0200
+++ vdr-2.3.9/menu.h    2018-04-08 12:34:13.683106263 +0200
@@ -224,7 +224,8 @@
   eOSState Sort(void);
   eOSState Commands(eKeys Key = kNone);
 protected:
-  cString DirectoryName(const cRecordings *Recordings);
+//  cString DirectoryName(const cRecordings *Recordings);
+  cString DirectoryName(void);
 public:
   cMenuRecordings(const char *Base = NULL, int Level = 0, bool OpenSubMenus = false, const cRecordingFilter *Filter = NULL);
   ~cMenuRecordings();

Display More

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

Diff

--- a/menu.c
+++ b/menu.c
@@ -823,7 +823,7 @@
   else {
      cStringList Dirs;
      for (const cRecording *Recording = Recordings->First(); Recording; Recording = Recordings->Next(Recording)) {
-         cString Folder = Recording->Folder();
+         cString Folder = Recording->FileFolder();
          strreplace((char *)*Folder, FOLDERDELIMCHAR, FOLDERDELIMCHARSUBST); // makes sure parent folders come before subfolders
          if (Dirs.Find(Folder) < 0)
             Dirs.Append(strdup(Folder));
@@ -2582,7 +2582,7 @@
   SetMenuCategory(mcRecordingEdit);
   recording = Recording;
   originalFileName = recording->FileName();
-  strn0cpy(folder, recording->Folder(), sizeof(folder));
+  strn0cpy(folder, recording->FileFolder(), sizeof(folder));
   strn0cpy(name, recording->BaseName(), sizeof(name));
   priority = recording->Priority();
   lifetime = recording->Lifetime();
@@ -2667,7 +2667,7 @@
 
 eOSState cMenuRecordingEdit::Folder(void)
 {
-  return AddSubMenu(new cMenuFolder(tr("Select folder"), &Folders, recording->Name()));
+  return AddSubMenu(new cMenuFolder(tr("Select folder"), &Folders, *recording->FullName()));
 }
 
 eOSState cMenuRecordingEdit::Action(void)
@@ -2748,7 +2748,7 @@
   cString OldFolder = Recording->Folder();
   cString NewName = *folder ? cString::sprintf("%s%c%s", folder, FOLDERDELIMCHAR, name) : name;
   NewName.CompactChars(FOLDERDELIMCHAR);
-  if (strcmp(NewName, Recording->Name())) {
+  if (strcmp(NewName, *Recording->FullName())) {
      if (!Recording->ChangeName(NewName)) {
         StateKey.Remove(Modified);
         Skins.Message(mtError, tr("Error while changing folder/name!"));
@@ -3068,6 +3068,16 @@
 {
   cString d(cVideoDirectory::Name());
   if (base) {
+     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;
+           }
+        }
+
      char *s = ExchangeChars(strdup(base), true);
      d = AddDirectory(d, s);
      free(s);

Display More

machtnix · April 8, 2018 at 4:44 PM

Quote from Saman

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

Diff

pi@raspberrypi:~/Entwicklung $ diff -ruN vdr-2.3.9++/menu.c vdr-2.3.9/menu.c
--- vdr-2.3.9++/menu.c    2018-03-26 20:16:28.134379878 +0200
+++ vdr-2.3.9/menu.c    2018-04-08 13:03:20.346681250 +0200
@@ -3009,7 +3009,8 @@
         CurrentRecording = *fileName ? *fileName : cReplayControl::LastReplayed();
      int current = Current();
      Clear();
-     GetRecordingsSortMode(DirectoryName(Recordings));
+//     GetRecordingsSortMode(DirectoryName(Recordings));
+     GetRecordingsSortMode(DirectoryName());
      Recordings->Sort();
      cMenuRecordingItem *CurrentItem = NULL;
      cMenuRecordingItem *LastItem = NULL;
@@ -3066,11 +3067,13 @@
   fileName = FileName;
 }
 
-cString cMenuRecordings::DirectoryName(const cRecordings *Recordings)
+//cString cMenuRecordings::DirectoryName(const cRecordings *Recordings)
+cString cMenuRecordings::DirectoryName()
 {
   cString d(cVideoDirectory::Name());
   if (base) {
      if (cVideoDirectory::HideFirstRecordingLevel()) {
+        LOCK_RECORDINGS_READ;
         cRecordings::cFolderInfos::cFolderInfo* info = Recordings->GetFolderInfo(base);
         if (info) {
            if (info->FirstFolderNames.Size() > 0)
@@ -3263,8 +3266,9 @@
      return osContinue;
   if (const cMenuRecordingItem *ri = (cMenuRecordingItem *)Get(Current()))
      SetRecording(ri->Recording()->FileName()); // makes sure the Recordings menu will reposition to the current recording
-  cRecordings *Recordings = cRecordings::GetRecordingsWrite(recordingsStateKey); // write access is necessary for sorting!
-  IncRecordingsSortMode(DirectoryName(Recordings));
+//  cRecordings *Recordings = cRecordings::GetRecordingsWrite(recordingsStateKey); // write access is necessary for sorting!
+//  IncRecordingsSortMode(DirectoryName(Recordings));
+  IncRecordingsSortMode(DirectoryName());
   recordingsStateKey.Reset();
   Set(true);
   return osContinue;
pi@raspberrypi:~/Entwicklung $ diff -ruN vdr-2.3.9++/menu.h vdr-2.3.9/menu.h
--- vdr-2.3.9++/menu.h    2018-03-26 17:20:13.376544110 +0200
+++ vdr-2.3.9/menu.h    2018-04-08 12:34:13.683106263 +0200
@@ -224,7 +224,8 @@
   eOSState Sort(void);
   eOSState Commands(eKeys Key = kNone);
 protected:
-  cString DirectoryName(const cRecordings *Recordings);
+//  cString DirectoryName(const cRecordings *Recordings);
+  cString DirectoryName(void);
 public:
   cMenuRecordings(const char *Base = NULL, int Level = 0, bool OpenSubMenus = false, const cRecordingFilter *Filter = NULL);
   ~cMenuRecordings();

Display More

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

Diff

--- a/menu.c
+++ b/menu.c
@@ -823,7 +823,7 @@
   else {
      cStringList Dirs;
      for (const cRecording *Recording = Recordings->First(); Recording; Recording = Recordings->Next(Recording)) {
-         cString Folder = Recording->Folder();
+         cString Folder = Recording->FileFolder();
          strreplace((char *)*Folder, FOLDERDELIMCHAR, FOLDERDELIMCHARSUBST); // makes sure parent folders come before subfolders
          if (Dirs.Find(Folder) < 0)
             Dirs.Append(strdup(Folder));
@@ -2582,7 +2582,7 @@
   SetMenuCategory(mcRecordingEdit);
   recording = Recording;
   originalFileName = recording->FileName();
-  strn0cpy(folder, recording->Folder(), sizeof(folder));
+  strn0cpy(folder, recording->FileFolder(), sizeof(folder));
   strn0cpy(name, recording->BaseName(), sizeof(name));
   priority = recording->Priority();
   lifetime = recording->Lifetime();
@@ -2667,7 +2667,7 @@
 
 eOSState cMenuRecordingEdit::Folder(void)
 {
-  return AddSubMenu(new cMenuFolder(tr("Select folder"), &Folders, recording->Name()));
+  return AddSubMenu(new cMenuFolder(tr("Select folder"), &Folders, *recording->FullName()));
 }
 
 eOSState cMenuRecordingEdit::Action(void)
@@ -2748,7 +2748,7 @@
   cString OldFolder = Recording->Folder();
   cString NewName = *folder ? cString::sprintf("%s%c%s", folder, FOLDERDELIMCHAR, name) : name;
   NewName.CompactChars(FOLDERDELIMCHAR);
-  if (strcmp(NewName, Recording->Name())) {
+  if (strcmp(NewName, *Recording->FullName())) {
      if (!Recording->ChangeName(NewName)) {
         StateKey.Remove(Modified);
         Skins.Message(mtError, tr("Error while changing folder/name!"));
@@ -3068,6 +3068,16 @@
 {
   cString d(cVideoDirectory::Name());
   if (base) {
+     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;
+           }
+        }
+
      char *s = ExchangeChars(strdup(base), true);
      d = AddDirectory(d, s);
      free(s);

Display More

Patch 1 scheint ok - zumindest kein Absturz mehr.

Danke

Saman · April 8, 2018 at 8:16 PM

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.

seahawk1986 · April 9, 2018 at 8:27 AM

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).

CKone · April 9, 2018 at 9:56 AM

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

zaubi4u · April 9, 2018 at 10:27 AM

Hallo Christian,

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

Gruß Micha

iNOB · April 9, 2018 at 10:44 AM

Quote from zaubi4u

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.

CKone · April 9, 2018 at 10:49 AM

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

Saman · April 9, 2018 at 12:55 PM

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

SurfaceCleanerZ · April 9, 2018 at 1:09 PM

Hi,

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

MfG Stefan

CKone · April 9, 2018 at 1:34 PM

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

Saman · April 9, 2018 at 1:41 PM

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.

Saman · April 10, 2018 at 1:07 PM

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");
}

Saman · April 10, 2018 at 7:25 PM

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.

Code

cRecordings::cFolderInfos::cFolderInfo *cRecordings::cFolderInfos::cFolderTree::GetInfo(void) const
{
  cFolderInfo *info = new cFolderInfo(*name, *FullName(), count, latest, *latestFileName);
  // take care that LOCALRECFOLDER is the first item
  bool addLocal = false;
  for (int i = 0; i < firstFolderNames.Size(); i++) {
      if (strcmp(firstFolderNames.At(i), LOCALRECFOLDER) == 0)
         addLocal = true;
      else
         info->FirstFolderNames.Append(strdup(firstFolderNames.At(i)));
      }
  info->FirstFolderNames.Sort();
  if (addLocal)
     info->FirstFolderNames.Insert(strdup(LOCALRECFOLDER));
  return info;
}

Display More

seahawk1986 · April 11, 2018 at 9:07 AM

Prima, damit klappt die Sortierung der Aufnahmen in Unterverzeichnissen wieder

Saman · April 11, 2018 at 9:28 AM

Quote from zaubi4u

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.

CKone · April 11, 2018 at 1:52 PM

dann kann die Version 2.4.0 ja zum WE kommen!

Vielen Dank für den Fix

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

Participate now!

Users Viewing This Thread