Skindesigner mit SVG-Rendering-Support

M-Reimer · 9. November 2014

Ich habe mich mal mit dem Thema "Vektorgrafik in VDR-Skins" befasst. Nachdem es mit dem Skindesigner ja nun eine äußerst praktische Lösung zum Skin-Schreiben für den VDR gibt, bietet es sich natürlich geradezu an diese Funktion dort zu platzieren, damit möglichst viele Skin-Entwickler davon profitieren können. Man spart sich mit eingebautem SVG-Support nicht nur das manuelle Umwandeln (falls man seine Icons mit Inkscape erstellt) sondern man profitiert auch davon, dass das Bild passend zur gewählten Auflösung gerendert wird. Unschärfen durch Skalieren treten somit keine mehr auf.

Wenn es um SVG im "Open-Source-Bereich" geht kommt man um Cairo in Kombination mit "librsvg" eigentlich nicht herum. Cairo selbst kann mit PNG als Rasterformat umgehen. JPEG für die Scraper-Bilder habe ich mit "libjpeg" zusätzlich nachgerüstet. Letztere ist eigentlich garkeine eigene Abhängigkeit, denn der VDR benötigt auch die libjpeg. Ich empfehle "libjpeg-turbo". Es sollte aber eigentlich jede gängige libjpeg gehen, denn spezielle Features der libjpeg-turbo nutze ich nicht.

Unterstützte Bildformate sind damit: PNG, SVG und JPG.

Die Abhängigkeit zu ImageMagick oder GraphicsMagick entfällt dabei komplett. Rastergrafik wird von Cairo skaliert, SVG wird über librsvg direkt über Cairo auf die gewünschte Zielgröße gerendert. Dabei wird auch Anti-Aliasing verwendet.

Ich pflege die angepasste Version aktuell in einem eigenen GIT-Repository. Entwickelt habe ich das ganze in Absprache (via PM) mit Louis. Ziel ist es, die Repositories, nachdem die eventuell neu reingekommen Bugs raus sind, wieder zusammenzuführen.

Bitte in diesem Thread nur Bug-Reports und Feature-Requests, die direkt mit dem Laden von Bildern (besonders SVG) zu tun haben. Ich werde nur Anpassungen und Fixes in diesem Bereich umsetzen. Alle anderen Änderungen pulle ich regelmäßig aus dem Repository von Louis.

Link zum Repository: https://github.com/M-Reimer/vdr-plugin-skindesigner/

louis · 9. November 2014

Moin,

coole Sache...vielen Dank an M-Reimer für die Unterstützung!

Testet mal fleissig, damit ich das bald ins offizielle Git aufnehmen kann

Ciao Louis

Copperhead · 9. November 2014

Getestet und für gut befunden.

louis · 9. November 2014

Zitat von Copperhead

Getestet und für gut befunden.

Jetzt kannst du endlich deine geliebten SVGs einsetzen Wann bekommt man eigentlich mal was Holo-technisches zu sehen?

Ciao Louis

Copperhead · 9. November 2014

Zitat von louis

Jetzt kannst du endlich deine geliebten SVGs einsetzen Wann bekommt man eigentlich mal was Holo-technisches zu sehen?

SkinDesigner - Holo

lostinspc · 9. November 2014

Hab's mal installiert. Funktioniert bislang so gut wie vorher.

Sichtbarer Unterschied bei metrixHd: In displaychannel werden die nicht aktiven Symbole (z.B. HD, REC und das ganz außen rechts deutlich dunkler bzw. mit mehr Transparenz als beim Original angezeigt. Finde ich persönlich allerdings eher besser, vorher konnte ich den Unterschied zwischen aktiv und nicht aktiv schlecht erkennen.

vectra130 · 9. November 2014

Stehe gerade aufm Schlauch. Bekomme folgende Fehlermeldung und weiß gerade nicht warum

Code

*** Plugin skindesigner:
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -I/usr/src/v4l-dvb/linux/include/uapi -c -DPLUGIN_NAME_I18N='"skindesigner"'  -I/usr/include/libxml2 -I/usr/local/src/vdr-git/include -I/usr/include/freetype2   -pthread -I/usr/include/librsvg-2.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/libpng12 -I/usr/include/pixman-1 -I/usr/include/freetype2   -ljpeg -o libcore/imageloader.o libcore/imageloader.c
libcore/imageloader.c: In member function ‘virtual void cImageImporterSVG::DrawToCairo(cairo_t*)’:
libcore/imageloader.c:212:44: error: ‘rsvg_handle_render_cairo’ was not declared in this scope
libcore/imageloader.c: In member function ‘virtual void cImageImporterJPG::DrawToCairo(cairo_t*)’:
libcore/imageloader.c:318:37: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
make[1]: *** [libcore/imageloader.o] Error 1

Gruß Patrick

Frodo · 9. November 2014

Ich bekomme das Plugin gegen die neuen Libraies nicht gebaut:

build.log

Code

libcore/../libcore/imageloader.h:59:1: warning: 'void my_error_exit(j_common_ptr)' defined but not used [-Wunused-function]
libcore/../libcore/imageloader.h:71:1: warning: 'void my_output_message(j_common_ptr)' defined but not used [-Wunused-function]
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -c -DPLUGIN_NAME_I18N='"skindesigner"'  -I/usr/include/libxml2 -I/usr/include/freetype2   -pthread -I/usr/include/librsvg-2.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/libpng12 -I/usr/include/pixman-1 -I/usr/include/freetype2   -ljpeg -o libcore/imagecache.o libcore/imagecache.c
libcore/imagecache.c: In member function 'cImage* cImageCache::GetLogo(std::string, int, int)':
libcore/imagecache.c:90:77: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
libcore/imagecache.c: In member function 'std::string cImageCache::GetIconName(std::string, eMenuCategory)':
libcore/imagecache.c:203:12: warning: enumeration value 'mcUndefined' not handled in switch [-Wswitch]
libcore/imagecache.c:203:12: warning: enumeration value 'mcUnknown' not handled in switch [-Wswitch]
libcore/imagecache.c:203:12: warning: enumeration value 'mcMain' not handled in switch [-Wswitch]
libcore/imagecache.c:203:12: warning: enumeration value 'mcText' not handled in switch [-Wswitch]
libcore/imagecache.c:203:12: warning: enumeration value 'mcFolder' not handled in switch [-Wswitch]
libcore/imagecache.c:203:12: warning: enumeration value 'mcCam' not handled in switch [-Wswitch]
libcore/imagecache.c: In member function 'bool cImageCache::LoadLogo(const cChannel*)':
libcore/imagecache.c:332:10: warning: unused variable 'success' [-Wunused-variable]
libcore/imageloader.h: At global scope:
libcore/imageloader.h:59:1: warning: 'void my_error_exit(j_common_ptr)' defined but not used [-Wunused-function]
libcore/imageloader.h:71:1: warning: 'void my_output_message(j_common_ptr)' defined but not used [-Wunused-function]
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -c -DPLUGIN_NAME_I18N='"skindesigner"'  -I/usr/include/libxml2 -I/usr/include/freetype2   -pthread -I/usr/include/librsvg-2.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/libpng12 -I/usr/include/pixman-1 -I/usr/include/freetype2   -ljpeg -o libcore/helpers.o libcore/helpers.c
libcore/helpers.c: In member function 'std::vector<std::basic_string<char> >& splitstring::split(char, int)':
libcore/helpers.c:153:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
libcore/../libcore/imageloader.h: At global scope:
libcore/../libcore/imageloader.h:59:1: warning: 'void my_error_exit(j_common_ptr)' defined but not used [-Wunused-function]
libcore/../libcore/imageloader.h:71:1: warning: 'void my_output_message(j_common_ptr)' defined but not used [-Wunused-function]
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -c -DPLUGIN_NAME_I18N='"skindesigner"'  -I/usr/include/libxml2 -I/usr/include/freetype2   -pthread -I/usr/include/librsvg-2.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/libpng12 -I/usr/include/pixman-1 -I/usr/include/freetype2   -ljpeg -o libcore/imageloader.o libcore/imageloader.c
libcore/imageloader.c: In member function 'virtual void cImageImporterSVG::DrawToCairo(cairo_t*)':
libcore/imageloader.c:212:44: error: 'rsvg_handle_render_cairo' was not declared in this scope
libcore/imageloader.c: In member function 'virtual void cImageImporterJPG::DrawToCairo(cairo_t*)':
libcore/imageloader.c:318:37: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
make[1]: *** [libcore/imageloader.o] Error 1

Alles anzeigen

Hat einer eine Idee was hier noch fehlt?

TheChief · 9. November 2014

Ins blaue geraten das dev Paket von librsvg?

vectra130 · 9. November 2014

Das ist installiert:

Spoiler anzeigen

Frodo · 9. November 2014

Die Pakete sind vorhanden:

Pakete

Ich sehe gerade unter Ubuntu Trusty baut das Plugin.
Das heißt mit Precise bzw. dem aktuellen YaVDR funktioniert es nicht, da dort die libcairo älter ist (precise: 1.10 bzw. trusty: 1.13) .

M-Reimer · 9. November 2014

libcairo ist nicht schuld sondern librsvg. Dort hat es in der Zwischenzeit leider eine "Regeländerung" gegeben. Man hat jetzt die Wahl zwischen

a) Doofer Warnung das man "rsvg-cairo.h" nicht direkt includen darf
b) Fehlende Rückwärtskompatibilität oder
c) Komischer Workaround der prüft ob ein "define" aus einem der von "rsvg.h" includeten Header-Files gesetzt ist.

Ich habe mich für c) entschieden. Fix ist im GIT.

Kann mir einer der YaVDR-Nutzer bitte seine librsvg-Version nennen?

Was die Warnings angeht: Die müssen natürlich auch noch raus. Wenn da also jemand mit dem aktuellen GIT-Stand noch irgendwas aus der Ecke "imageloader.h" oder "imageloader.c" bekommen sollte, dann immer her damit.

M-Reimer · 9. November 2014

Zitat von lostinspc

Sichtbarer Unterschied bei metrixHd: In displaychannel werden die nicht aktiven Symbole (z.B. HD, REC und das ganz außen rechts deutlich dunkler bzw. mit mehr Transparenz als beim Original angezeigt. Finde ich persönlich allerdings eher besser, vorher konnte ich den Unterschied zwischen aktiv und nicht aktiv schlecht erkennen.

Das "HD" und das "Wide" habe ich bereits neu gemacht (liegt als SVG im metrixHD-Skin). Das Original war nach dem Skalieren sichtbar verwaschen. Mit der SVG kann das nicht mehr passieren.

Für die Skin-Entwickler: Wenn SVG und PNG parallel liegen, dann wird SVG bevorzugt verwendet.

Und noch als allgemeine Info: Die gerenderten SVGs landen in dem Cache, den Louis schon für skalierte Rastergrafiken vorgesehen hatte. Es wird also nur einmal gerendert, was der Performance zu gute kommt.

vectra130 · 9. November 2014

Cool. Jetzt baut es durch. Danke Dir

Frodo · 9. November 2014

M-Reimer
Bei YaVDR 0.5 bzw. Ubuntu Precise 12.04 hat die librsvg die Version 2.36.1 .

Mit deinem letzten Fix baut es unter Precise noch immer nicht, ich habe jetzt die Cairo Version 1.13 von Trusty nach Precise portiert mal sehen ob es damit klappt.

M-Reimer · 9. November 2014

Danke. Die Version, ab der die neuen Regeln gelten, ist 2.36.2. Somit ist YaVDR da knapp drunter.

Welchen Fehler bekommst du denn? Der ausschlaggebende Fehler bei dir war genau der, den ich gerade versucht habe zu fixen. Hast du mal "make clean" versucht?

Frodo · 9. November 2014

Danke für den Tipp, nach einem "make clean" bzw. "debclean" baut es auch mit Cairo 1.10

M-Reimer · 10. November 2014

Ich habe mal alle verbleibenden Compiler-Warnings gefixt und dafür gesorgt, dass Debug-Meldungen aus dem imageloader-Code nurnoch im Syslog landen wenn das auch im Setup so konfiguriert ist.

Allerdings frage ich mich ob man da nicht ohnehin anders vorgehen sollte.

Echte Fehler beim Bild-Laden sollten meiner Ansicht nach immer geloggt werden. Und zwar nicht mit "dsyslog" sondern korrekt mit "esyslog".

Frodo · 10. November 2014

Mit dem Holo Theme crashed bei mir der VDR alle anderen Themes funktionieren. Ich habe meine Logs bereits hier abgelegt, weis aber nicht ob die hilfreich sind.

M-Reimer · 10. November 2014

Für mich nicht. Das wäre jetzt der richtige Zeitpunkt dafür, dass hier mal jemand aufschlägt, der sich mit sowas auskennt...

Mit c++filt kann man das zwar etwas lesbarer machen:

Code

/lib/x86_64-linux-gnu/libc.so.6(+0x7e846)[0x7fc739716846]
/lib/x86_64-linux-gnu/libc.so.6(+0x7ebb0)[0x7fc739716bb0]
/lib/x86_64-linux-gnu/libc.so.6(+0x7f839)[0x7fc739717839]
/usr/bin/vdr(cPixmapMemory::~cPixmapMemory()+0x1b)[0x4cdb1b]
/usr/bin/vdr(cPixmapMemory::~cPixmapMemory()+0x9)[0x4cdb39]
/usr/bin/vdr(cOsd::DestroyPixmap(cPixmap*)+0x133)[0x4d2f33]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cPixmapContainer::~cPixmapContainer()+0x5a)[0x7fc726f5d51a]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cDisplayMenuItemCurrentMainView::~cDisplayMenuItemCurrentMainView()+0x9)[0x7fc726fe9469]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cDisplayMenuItemView::~cDisplayMenuItemView()+0x28)[0x7fc726fe1948]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cDisplayMenuItemMainView::~cDisplayMenuItemMainView()+0x9)[0x7fc726fe1b69]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cDisplayMenuListView::Clear()+0x2c)[0x7fc726fd4efc]
/usr/lib/vdr/plugins/libvdr-skindesigner.so.2.1.6(cDisplayMenuRootView::Clear()+0x15)[0x7fc726fc2775]
/usr/bin/vdr(cOsdMenu::Display()+0x7f)[0x4ccb1f]
/usr/bin/vdr(cMenuMain::Set()+0x24d)[0x4c05cd]
/usr/bin/vdr(cMenuMain::ProcessKey(eKeys)+0x37c)[0x4c23ac]
/usr/bin/vdr(main+0x2160)[0x4714e0]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xed)[0x7fc7396b976d]
/usr/bin/vdr[0x473475]

Alles anzeigen

Aber weiterhelfen tut mir das auch nicht. Keine Ahnung ob meine Änderungen hier überhaupt etwas mit dem Crash zu tun haben.

Skindesigner mit SVG-Rendering-Support

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema