[skindesigner] Aktuelle Version

kamel5 · May 29, 2021 at 9:47 PM

Das brauchst Du nicht umstellen. Ich lade das im Anschluss noch hoch.

Grüße

kamel5

seahawk1986 · May 30, 2021 at 12:04 AM

Ein Paket mit dem aktuellen Git-Stand ist im PPA veröffentlicht.

kamel5 · May 31, 2021 at 4:41 PM

Ich habe eine neue Version 1.2.17 angelegt.

Diese Version sollte jetzt etwas länger beständig sein, da mir im Moment keine (bis auf eins) weiteren offensichtlichen Probleme bekannt sind.

Das eine Problem bezieht sich auf das "Einfahren von links (shifttype="left")", z.B. beim Zapcockpit. Hier ist es so, das das Einfahren der Kanalliste bei mir nicht funktioniert. Das Einfahren von Elementen von Rechts funktioniert übrigens.

Ich habe deshalb mit dem letzten commit den default von shifttimezapcockpit vom skin estuary4vdr auf 0 gestellt. Dieser Wert kann auch im Setup geändert werden. Damit werden die genannten Elemente vom Zapcockpit angezeigt, aber eben nicht eingefahren. In wie weit andere Skins davon betroffen sind, habe ich nicht überprüft.

Der Code, der diese Animationen macht ist sehr komplex. Da ich das Ganze noch nicht durchschaut habe, fehlt mir da im Moment ein wenig der Antrieb, mich damit zu beschäftigen. Vielleicht später einmal. Ich würde das jetzt einfach mal offen lassen.

Grüße

kamel5

9000H · May 31, 2021 at 4:54 PM

Hi,

mit dem neuen git bekomme ich so was

Display Spoiler

Code

coreengine/viewdetail.c: In member function 'virtual bool cViewDetailRec::Parse(bool)':
coreengine/viewdetail.c:679:31: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
  679 |         errors = info->Errors();
      |                               ^
In file included from /usr/local/include/vdr/skins.h:18,
                 from /usr/local/include/vdr/osdbase.h:15,
                 from coreengine/../services/epgsearch.h:31,
                 from coreengine/viewdetail.h:4,
                 from coreengine/viewdetail.c:1:
/usr/local/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   92 |   int Errors(void) { return errors; }
      |       ^~~~~~
make: *** [Makefile:142: coreengine/viewdetail.o] Error 1
coreengine/listelements.c: In member function 'virtual bool cLeMenuRecordings::Parse(bool)':
coreengine/listelements.c:2062:27: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
 2062 |     errors = info->Errors();
      |                           ^
In file included from /usr/local/include/vdr/skins.h:18,
                 from /usr/local/include/vdr/osdbase.h:15,
                 from /usr/local/include/vdr/menuitems.h:15,
                 from /usr/local/include/vdr/plugin.h:14,
                 from coreengine/globals.h:14,
                 from coreengine/viewelement.h:11,
                 from coreengine/listelements.h:4,
                 from coreengine/listelements.c:1:
/usr/local/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   92 |   int Errors(void) { return errors; }
      |       ^~~~~~
coreengine/listelements.c: In member function 'virtual bool cCeMenuRecordings::Parse(bool)':
coreengine/listelements.c:2282:27: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
 2282 |     errors = info->Errors();
      |                           ^
In file included from /usr/local/include/vdr/skins.h:18,
                 from /usr/local/include/vdr/osdbase.h:15,
                 from /usr/local/include/vdr/menuitems.h:15,
                 from /usr/local/include/vdr/plugin.h:14,
                 from coreengine/globals.h:14,
                 from coreengine/viewelement.h:11,
                 from coreengine/listelements.h:4,
                 from coreengine/listelements.c:1:
/usr/local/include/vdr/recording.h:92:7: note:   in call to 'int cRecordingInfo::Errors()'
   92 |   int Errors(void) { return errors; }
      |       ^~~~~~
make: *** [Makefile:141: coreengine/listelements.o] Error 1

Display More

CU

9000h

kamel5 · May 31, 2021 at 5:33 PM

9000H ,

das ist hier VDR Developer Version 2.5.4 schon mal angesprochen worden.

Ich hoffe, das das kls noch so ändert.

Du kannst ja das const in recordings.h vorerst selbst einfügen.

Grüße

kamel5

cinfo · June 1, 2021 at 8:47 PM

aktuelle Version aus dem Git kann man nicht am VDR-2.5.4 erstellen

es ist immer der gleiche Fehler in div. Dateien -- kann man das nicht ändern?

Code

coreengine/viewelementsdisplaymenu.c:1234:31: error: passing 'const cRecordingInfo' as 'this' argument discards qualifiers [-fpermissive]
 1234 |         errors = info->Errors();
      |                               ^

diese Änderung hatte ich vorab gemacht

Quote

das ist hier VDR Developer Version 2.5.4 schon mal angesprochen worden.

Ich hoffe, das das kls noch so ändert.

Du kannst ja das const in recordings.h vorerst selbst einfügen.

Grüße

cinfo

kamel5 · June 1, 2021 at 9:10 PM

cinfo , ich nutze das so mit angepaßstem VDR-2.5.4.

Hast Du tatsächlich die Änderung in recording.h gemacht?

Denn ohne diese Änderung kommt genau diese Fehlermeldung.

Und dann komplett alles neu übersetzen.

Gr0ße

kamel5

cinfo · June 1, 2021 at 9:30 PM

Code

/usr/include/vdr/recording.h:94:7: note:   in call to 'int cRecordingInfo::Errors()'
   94 |   int Errors(void) { return errors; } // returns -1 if undefined

Hm, sieht so aus das sie eingetragen ist

cinfo

kamel5 · June 1, 2021 at 9:43 PM

Nein, ist nicht eingetragen.

Die Zeile soll so aussehen:

94 | int Errors(void) const { return errors; } // returns -1 if undefined

Auf das const in der Mitte kommt es an.

Dann aber auch den VDR komplett neu übersetzen.

Ich sollte wohl morgen noch einen Patch dazu machen.

Grüße

kamel5

cinfo · June 1, 2021 at 9:49 PM

danke jetzt geht es

Grüße

cinfo

kamel5 · June 3, 2021 at 2:27 PM

Nachtrag zur Version 1.2.17:

Damit diese Version mit VDR-2.5.4 übersetzt werden kann, ist zwingend der angefügte Patch auf den Core-VDR-2.5.4 anzuwenden.

Für VDR Versionen < 2.5.4 ist dieser Patch nicht nötig.

Da sich dieser Patch auf eine mit VDR-2.5.4 neu hinzugefügte Funktion bezieht, gibt es keinerlei sonstige Abhängigkeiten.

Der aktuelle Patch findet sich hier:

[skindesigner] Aktuelle Version

Grüße

kamel5

M-Reimer · June 6, 2021 at 3:22 PM

Ist es möglich das im Skindesigner zu patchen? Oder muss kls hier mal einen Blick drauf werfen?

kamel5 · June 6, 2021 at 4:13 PM

M-Reimer , das ist schon bekannt und soll in der nächsten Version mit einfließen:

VDR Developer Version 2.5.4

Grüße

kamel5

wirbel · June 6, 2021 at 4:44 PM

Ich denke, das Problem kann in im Plugin selbst problemlos gefixed werden.

kamel5 · June 6, 2021 at 5:41 PM

Quote from wirbel

Ich denke, das Problem kann in im Plugin selbst problemlos gefixed werden.

Das mag schon sein. Ich habe mich dann aber bewusst dagegen entschieden, für eine einzige frühe Entwicklerversion einen Sonderfall einzuführen.

Ich dachte, wer schon eine Entwicklerversion einsetzt, kann auch diesen Patch anwenden. Das gilt es wohl dann zu überdenken.

OK, wenn das der allgemeine Wunsch ist, dann gibt es demnächst noch einen commit dazu.

Grüße

kamel5

M-Reimer · June 6, 2021 at 5:59 PM

Edit: Nicht richtig gelesen: Wenn es von kls ein Update geben wird, dann würde ich das tatsächlich auch nicht im Plugin fixen.

wirbel · June 6, 2021 at 6:02 PM

Wenn ihr ohne Patch in einem (unwichtigem) Plugin auskommen könnt und der Entwicklung von VDR folgen, dann wäre das eine sehr gute Idee.

kamel5 · June 6, 2021 at 6:04 PM

Es gibt einen commit im git für alle, die den VDR nicht patchen wollen.

Alle Anderen können bei Version 1.2.17 bleiben.

Grüße

kamel5

kamel5 · June 6, 2021 at 6:14 PM

Quote from wirbel

Wenn ihr ohne Patch in einem (unwichtigem) Plugin auskommen könnt und der Entwicklung von VDR folgen, dann wäre das eine sehr gute Idee.

Naja, wenn der Patch von kls selbst gekommen ware, gäbe es sicher diese Diskussion nicht...

Ansonsten lehrt das eigentlich nur, das man mit neuen Funktionen warten sollte, bis dann tatsächlich alles im Core-VDR korrekt ist.

Grüße

kamel5

kamel5 · June 7, 2021 at 12:01 PM

Nachdem ich jetzt nochmal etwas länger über den letzten commit nachgedacht habe und der mir immer weniger gefallen hat, habe ich nach einer etwas dauerhafteren Lösung gesucht. Den letzte commit habe ich jetzt gegen einen neuen ausgetauscht.

Grundsätzlich habe ich die Sonderlösung für VDR-2.5.4 wieder entfernt und die Anzeige von Fehlern erst ab VDR-2.5.5 aktiviert.

Damit kann das skindesigner-Plugin auch ohne einen Patch übersetzt werden. Es fehlt dann aber die Anzeige der TS-Errors.

Wer mit VDR-2.5.4 die TS-Errors anzeigen will (wie beispielhaft in estuary4vdr eingebaut), muss den etwas erweiterten beigefügten Patch auf den Core-VDR anwenden, dadurch werden die TS-Errors wieder aktiviert. Ein anschließendes Neuübersetzen des Plugins nicht vergessen.

Sorry für das hin und her.

Grüße

kamel5

[skindesigner] Aktuelle Version

Participate now!

Tags

Users Viewing This Thread