[Announce] libSkindesignerAPI

louis · April 6, 2015 at 1:46 PM

Moin,

Quote from M-Reimer

Eben doch! Du holst Header-Dateien vom VDR in deine Library. Damit wird sie mit einem VDR-Update potentiell inkompatibel und muss zusätzlich neu kompiliert werden. Ein Grund mehr das mit dem Plugin "Skindesigner" auszuliefern. Zumindest bei vdr4arch gibt es einen Automatismus um alle Plugins bei einem VDR-Update zu versionieren um einen kompletten Rebuild der Plugins auszulösen.

Die Lib ist kein Plugin. Es ist korrekt, dass die Lib bei einem VDR Update neu gebaut werden sollte. Aber das kann man doch auch unabhängig von den Plugins machen. Erst den VDR, dann die Lib, dann die Plugins, und zwar egal in welcher Reihenfolge. Das wäre der saubere Weg. Die Lib muss eigentlich nur in zwei Fällen neu gebaut werden: entweder die Lib selbst hat sich geändert oder der VDR wurde erneuert.

Ciao Louis

mini73 · April 6, 2015 at 1:46 PM

Moin!

Ich arbeite gerade an einem Patch für skindesigner, so dass der skindesigner dann die lokale lib aus dem Source-Verzeichnis nutzt und man erst mal alles bauen und anschließend installieren kann. Dann kann man das auch leichter in zwei (Installations-)Pakete aufteilen.

Ich teste gerade noch ein bisschen, kommt aber gleich, natürlich nur als Diskussionsgrundlage gedacht.

Lars.

helau · April 6, 2015 at 1:49 PM

Quote from Copperhead

Deswegen habe ich stillschweigend dein Makefile für meine Zwecke repariert.

Reparieren muss man nur Code welcher nicht baut, so wie ich damals den Extensionpatch, nachdem Du Ihn in den Haenden hattest
Das Makefile hier hingegen baut , evtl nicht in deiner Distribution dann muss Du es eben "anpassen" was Du ja auch gemacht hast ...

Copperhead · April 6, 2015 at 1:57 PM

Quote from louis

d.h. du musst darauf achten, dass der Skindesigner vor den beiden anderen Plugins installiert wird, das ist doch auch blöd. Wenn du die Lib zuerst (falls überhaupt notwendig) eigenständig bauen würdest, wäre die Reihenfolge der Installation der Plugins völlig egal.

tvguideng und weatherforecast haben eine Abhängigkeit auf das skindesigner Plugin. das geht sozusagen automatisch.

Quote from louis

Ok, wie würdest du es denn machen wenn die Lib in einem eigenen Git wäre?

Dann wäre ich dazu gezwungen ein eigenes Paket zu machen. Arch Developer Regeln und so. Es ist einfach so, dass ein Quellpaket erstmal komplett durchgebaut wird und dann erst am Ende entschieden wird, ob alles in einem Paket bleibt, oder in mehrere aufgeteilt wird. Daran kann ich auch nicht wirklich etwas ändern.

Quote from louis

Wenn nur der Skindesigner (ohne Änderungen an der Lib) aktualisiert werden soll, baust du die Lib trotzdem.

Ja, zwangsläufig, weil eben das gesamte Quellpaket in einem Rutsch gebaut wurde.

Quote from louis

Du solltest anderen ihre Meinung zugestehen und auch in Erwägung ziehen, dass diese Meinung trotz der Tatsache, dass sie nicht deiner eigenen entspricht, durchaus sinnvoll sein kann. Weiterhin solltest du in der Lage sein, sowas auch mal abzuhaken. Das zeugt von Größe Sorry wenn das jetzt etwas von oben herab rüberkommt, aber mit über 20 Jahren mehr Lebenserfahrung sollte man das auch mal dürfen.

Ich akzeptiere deine Meinung natürlich. Bitte nicht falsch verstehen.
Ich verstehe deine Beweggründe zwar immernoch nicht, aber mir ist es mittlerweile auch egal, wie es gemacht ist. Es ist natürlich schade, das so schöne Skins, wie z.B. skinflat ausgeschlossen werden.

Copperhead · April 6, 2015 at 1:58 PM

Quote from helau

Reparieren muss man nur Code welcher nicht baut, so wie ich damals den Extensionpatch, nachdem Du Ihn in den Haenden hattest

Danke für die Blumen Ja, heute betrachtet, hätte ich das etwas strukturierter angehen sollen.

mini73 · April 6, 2015 at 2:17 PM

Quote from 3PO

Code

PCDIR ?= $(shell pkg-config --variable pc_path pkg-config |cut -d ':' -f1)

Das funktioniert hier leider gar nicht, dadurch hat man ja gar keine Kontrolle mehr darüber, wo die pc-Datei installiert wird.

Bei mir sieht das so aus:

Code

$ pkg-config --variable pc_path pkg-config
/usr/local/lib/x86_64-linux-gnu/pkgconfig:/usr/local/lib/pkgconfig:/usr/local/share/pkgconfig:/usr/lib/x86_64-linux-gnu/pkgconfig:/usr/lib/pkgconfig:/usr/share/pkgconfig

D.h. es wird immer an eine Stelle installiert, wo ich es evtl. gar nicht haben will. Wenn ich die lib nach /usr installieren möchte, sollte die pc-Datei auch dort liegen.

Unter Ubuntu scheint pkg-config also per default so konfiguriert zu sein, dass es pc-Dateien unter /usr/local und /usr findet, und /usr/local natürlich zuerst, weil da ein Benutzer seinen Kram installieren darf/soll, der nicht vom Paketmanager abgedeckt wird.
Wenn ich aber das Plugin/die lib in ein Paket verpacke, muss ich es nach /usr installieren.

Ich bin dafür, es wieder wie vorher zu machen:

Code

PCDIR  ?= $(PREFIX)/lib/pkgconfig

Dann kann jeder es über die Build-Umgebung festlegen, wo es hin installiert werden soll.

Rest kommt gleich, bin mit den Tests gleich durch.

(und bitte nicht wieder in Diskussionen wie "Gentoo ist besser als Ubuntu" oder "Ubuntu ist ja auch doof" verstricken, zusammen sollten wir einen Weg finden, der in jeder Distribution ohne große Klimmzüge funktioniert)

Lars.

mini73 · April 6, 2015 at 2:21 PM

Leider gibt es "PREFIX" nicht in der vdr.pc, da gibt es nur direkt das Plugin-Lib-Verzeichnis usw.
D.h. für sowas wie libskindesignerapi kommt man nicht drumrum, PREFIX manuell beim Bauen zu setzen. Es sei denn, Klaus erweitert die vdr.pc noch z.B. mit VDR_PREFIX, wo PREFIX aus der Make.config drin gespeichert wird.

Lars.

mini73 · April 6, 2015 at 2:35 PM

So, hier mal mein Vorschlag für das Makefile (unten als Anhang):

Diff

diff --git a/Makefile b/Makefile
index f7cec16..d7db2cb 100644
--- a/Makefile
+++ b/Makefile
@@ -54,10 +54,6 @@ LIBS += $(shell pkg-config --libs librsvg-2.0 cairo-png) -ljpeg
 
 LIBS += $(shell xml2-config --libs)
 
-INCLUDES += $(shell pkg-config --cflags libskindesignerapi)
-LIBS += $(shell pkg-config --libs libskindesignerapi)
-DEFINES += -DLIBSKINDESIGNERAPIVERSION='"$(shell pkg-config --modversion libskindesignerapi)"'
-
 ### The object files:
 OBJS = $(PLUGIN).o \
        config.o \
@@ -113,12 +109,32 @@ OBJS = $(PLUGIN).o \
 
 ### The main target:
 
-all: $(SOFILE) i18n
+all: subprojects $(SOFILE) i18n
+
+### subprojects:
+
+.PHONY: libskindesignerapi subprojects
+
+libskindesignerapi:
+	$(MAKE) -C libskindesignerapi
+
+subprojects: libskindesignerapi
+
+install-subprojects:
+	$(MAKE) -C libskindesignerapi install
+
+clean-subprojects:
+	$(MAKE) -C libskindesignerapi clean
+
+### get version of subprojects in specific targets when subprojects have been built
+
+%.o: SUB_DEFINES = -DLIBSKINDESIGNERAPIVERSION='"$(shell pkg-config --modversion libskindesignerapi/libskindesignerapi.pc)"'
+$(SOFILE): SUB_LIBS = libskindesignerapi/libskindesignerapi.so.$(shell pkg-config --modversion libskindesignerapi/libskindesignerapi.pc)
 
 ### Implicit rules:
 
 %.o: %.c
-	$(CXX) $(CXXFLAGS) -c $(DEFINES) $(INCLUDES) -o $@ $<
+	$(CXX) $(CXXFLAGS) -c $(DEFINES) $(SUB_DEFINES) $(INCLUDES) -o $@ $<
 
 ### Dependencies:
 
@@ -158,7 +174,7 @@ install-i18n: $(I18Nmsgs)
 ### Targets:
 
 $(SOFILE): $(OBJS)
-	$(CXX) $(CXXFLAGS) $(LDFLAGS) -shared $(OBJS) $(LIBS) -o $@
+	$(CXX) $(CXXFLAGS) $(LDFLAGS) -shared $(OBJS) $(LIBS) $(SUB_LIBS) -o $@
 
 install-lib: $(SOFILE)
 	install -D $^ $(DESTDIR)$(LIBDIR)/$^.$(APIVERSION)
@@ -177,7 +193,7 @@ install-scripts:
 	mkdir -p $(DESTDIR)$(SKINDESIGNER_SCRIPTDIR)
 	cp -r scripts/* $(DESTDIR)$(SKINDESIGNER_SCRIPTDIR)
 
-install: install-lib install-i18n install-themes install-skins install-scripts
+install: install-subprojects install-lib install-i18n install-themes install-skins install-scripts
 
 dist: $(I18Npo) clean
 	@-rm -rf $(TMPDIR)/$(ARCHIVE)
@@ -187,6 +203,6 @@ dist: $(I18Npo) clean
 	@-rm -rf $(TMPDIR)/$(ARCHIVE)
 	@echo Distribution package created as $(PACKAGE).tgz
 
-clean:
+clean: clean-subprojects
 	@-rm -f $(PODIR)/*.mo $(PODIR)/*.pot
 	@-rm -f $(OBJS) $(DEPFILE) *.so *.tgz core* *~
diff --git a/displaymenu.h b/displaymenu.h
index 5a3fdb6..5b697b4 100644
--- a/displaymenu.h
+++ b/displaymenu.h
@@ -1,6 +1,6 @@
 #ifndef __DISPLAYMENU_H
 #define __DISPLAYMENU_H
-#include <libskindesignerapi/skindesignerapi.h>
+#include "libskindesignerapi/skindesignerapi.h"
 #include "libtemplate/template.h"
 #include "views/displaymenurootview.h"
 
diff --git a/displayplugin.h b/displayplugin.h
index c713a73..092c0a9 100644
--- a/displayplugin.h
+++ b/displayplugin.h
@@ -1,7 +1,7 @@
 #ifndef __DISPLAYPLUGIN_H
 #define __DISPLAYPLUGIN_H
 
-#include <libskindesignerapi/skindesignerapi.h>
+#include "libskindesignerapi/skindesignerapi.h"
 #include "libtemplate/template.h"
 #include "views/displaypluginview.h"
 
diff --git a/libskindesignerapi/Makefile b/libskindesignerapi/Makefile
index 7c24db8..fa1ce56 100644
--- a/libskindesignerapi/Makefile
+++ b/libskindesignerapi/Makefile
@@ -12,7 +12,7 @@ TARGET_LIB = $(SONAME).$(MINOR)
 PREFIX ?= /usr/local
 INCDIR ?= $(PREFIX)/include
 LIBDIR ?= $(PREFIX)/lib
-PCDIR  ?= $(shell pkg-config --variable pc_path pkg-config |cut -d ':' -f1)
+PCDIR  ?= $(PREFIX)/lib/pkgconfig
 TMPDIR ?= /tmp
 
 ### The name of the distribution archive:
diff --git a/skindesigner.c b/skindesigner.c
index 0398c7b..2deac2c 100644
--- a/skindesigner.c
+++ b/skindesigner.c
@@ -7,7 +7,7 @@
  */
 #include <getopt.h>
 #include <vdr/plugin.h>
-#include <libskindesignerapi/skindesignerapi.h>
+#include "libskindesignerapi/skindesignerapi.h"
 
 #define DEFINE_CONFIG 1
 #include "config.h"

Display More

Die pkg-config-Aufrufe für die libskindesignerapi hab ich an Stellen ausgelagert, wo sie gebraucht werden und wo dann auch sichergestellt ist, dass die lib schon gebaut wurde. Und es wird dann auch die pc-Datei aus dem Unterverzeichnis benutzt und nicht irgend eine Datei aus dem System, die ggf. zu einer älteren Version gehört.

Der skindesigner selbst benutzt immer die lokale lib aus dem Sourceverzeichnis selbst, braucht also keine installierte lib im System. Das macht die Entwicklung am Plugin selbst dann auch einfacher. Sonst müsste man ja nach jeder Änderung an der lib diese erst installieren, bevor man am skindesigner arbeiten kann. Das ist nicht praktikabel.

Beim "make" wird dann zuerst die lib gebaut, bei "make install" wird sie auch mit installiert. Je nach Umgebung muss man dann beim Installieren das PREFIX angeben, wie man das in normalen Build-Umgebungen eben macht, also ggf. "PREFIX=/usr make install" aufrufen. Natürlich könnte man irgendeine Variable aus der vdr.pc nehmen (z.B. bindir oder libdir) und daraus das PREFIX erraten, aber wirklich sicher ist das nicht. Schöner wäre evtl. sowas:

Diff

diff --git a/Makefile b/Makefile
index 9722036..9e83e8a 100644
--- a/Makefile
+++ b/Makefile
@@ -147,6 +147,7 @@ make-libsi: # empty rule makes sure the sub-make for libsi is always called


 .PHONY: vdr.pc
 vdr.pc:
+       @echo "vdr_prefix=$(PREFIX)" > $@
        @echo "bindir=$(BINDIR)" > $@
        @echo "mandir=$(MANDIR)" >> $@
        @echo "videodir=$(VIDEODIR)" >> $@

Display More

Lars.

Copperhead · April 6, 2015 at 2:54 PM

Ich sehe was du gemacht hast. Ich weiß aber noch nicht so richtig, worauf du hinaus willst.
Dieser Patch macht meine Änderungen unnötig.

Was ich allerdings immernoch nicht verstehe ist, warum ich die libskindesigner in ein eigenes Paket stecken soll.
Damit tvguideng funktioniert braucht es immer sowohl die libskindesigner, als auch das skindesigner-Plugin.

mini73 · April 6, 2015 at 3:01 PM

Das musst du nicht. Unter Debian ist es aber durchaus üblich. Und jetzt ist es vernünftig möglich, wenn man es möchte.

Lars.

louis · April 6, 2015 at 3:47 PM

Hi Lars,

danke für den Patch. Die jetzige Variante ist wohl doch sinniger, das erleichtert das Handling. Ich war zu sehr darauf fixiert, die Lib als eigenständig zu betrachten. Ich habe den Patch ins Git übernommen und eine Version 0.4.2 getaggt, damit man das auch vernünftig ausprobieren kann. Ausserdem habe ich das README angepasst - die Lib ist jetzt offiziell Teil des Skindesigners

Code

Installation
------------


Skindesigner consists of the Skindesigner Plugin itself and a shared library
called "libSkindesignerAPI" which allows other Plugins to use the facilities of
Skindesigner. Since these other Plugins need to have access to the library,
the library has to be proper installed on your system. 


With a "make install" both the plugin and the library are installed on your 
system. The destination where the library will be installed can be set with 
the PREFIX parameter: "PREFIX=/usr make install". Default of PREFIX is
/usr/local.  


After installation check if libskindesignerapi.pc is correctly found in your
PKG_CONFIG_PATH. Other Plugins using the library issue the following 
commands in their makefiles which should also work on your commandline:
pkg-config --cflags libskindesignerapi 
pkg-config --libs libskindesignerapi
pkg-config --modversion libskindesignerapi


Note that skindesigner itself and all plugins which use libSkindesignerAPI have to
be build against the same major verison of the library to be compatible.
...

Display More

Das muss natürlich nicht die entgültige Version sein...wenn sich in der weiteren Diskussion herausstellt, dass noch etwas verbesserungswürdig ist und ein Konsens besteht, übernehme ich das natürlich. Wie Lars schon geschrieben hat, es geht hier nicht um irgendwelche Distributionsthematiken, sondern darum, einen besten gemeinsamen Nenner zu finden.

Was ich wohl auch noch einbauen werde, dass bei gesetztem "LCLBLD" die benutzten Pfade der Lib komplett individuell konfigurierbar sind...es gibt doch noch einige "ich will alles in einem Verzeichnis haben" Anhänger.

Ciao Louis

mini73 · April 6, 2015 at 3:58 PM

Vielen Dank! Ich denke auch, dass wir zusammen da schon was passendes finden werden.

Lars

mini73 · April 6, 2015 at 4:58 PM

Ich muss demnächst dann noch mal testen, ob noch alles vernünftig neu gebaut wird, wenn man z.B. bei einem libskindesignerapi-Header etwas ändert. Das hab ich noch vergessen, komme heute aber nicht mehr dazu.

Lars

eurofinder · April 6, 2015 at 5:37 PM

Habe gerade eine Aktualisierung aus dem GIT gemacht- jetzt läuft es.

Gruß
Eurofinder

Frodo · April 6, 2015 at 7:11 PM

Local bauen funktioniert nun.

Als Ubuntu Paket klappt es bei mir noch nicht

Code

dh_usrlocal: debian/libskindesignerapi/usr/local/include/libskindesignerapi/osdelements.h is not a directory
dh_usrlocal: debian/libskindesignerapi/usr/local/include/libskindesignerapi/skindesignerosdbase.h is not a directory
dh_usrlocal: debian/libskindesignerapi/usr/local/include/libskindesignerapi/skindesignerapi.h is not a directory
rmdir: konnte »debian/libskindesignerapi/usr/local/include/libskindesignerapi“ nicht entfernen: Das Verzeichnis ist nicht leer
dh_usrlocal: rmdir debian/libskindesignerapi/usr/local/include/libskindesignerapi returned exit code 1
make: *** [binary] Fehler 1
dpkg-buildpackage: Fehler: Fehler-Exitstatus von debian/rules binary war 2

mini73 · April 6, 2015 at 7:21 PM

Zum einen würde ich PREFIX=/usr in die rules schreiben, zum anderen müsste ich mir erst mal die install-Dateien in deinem Paket ansehen. Komme ich aber erst Mittwoch zu.

Lars

marco · April 6, 2015 at 7:22 PM

Quote from M-Reimer

Wo soll der Vorteil sein, die Library als eigenes Paket zu haben? Keines der Plugins, das aktuell auf der "libskindesigner" aufbauen soll, funktioniert ohne das Skindesigner-Plugin selbst. Oder habe ich da etwas übersehen?

Eine Library, die ihrerseits auf VDR-Source-Bestandteilen aufbaut... Das musste für Probleme sorgen. Schon weil es beim VDR eben verschiedene Arten gibt, etwas zu bauen. Immerhin kann keiner sagen das ich euch da nicht weit im Voraus gewarnt hätte

Weil einige Distributionen eine Politik haben, daß gebündelte Bibliotheken nicht mit ausgelierfert werden dürfen.
Siehe auch Fedora unbundle Politik und
Debian/Ubuntu unbundle Politik

ohne die libskindesignerapi als separate lib zu kompilieren, würde es eben kein vdr-skindesigner/weatherforcast-Paket für Fedora geben, und ich habe halt gerne den VDR unter Fedora.

Was für ein Mist, sehe gerade, daß sich vdr-skindesigner-0.4.2 (mit der 0.4.1 vor der Änderung des Makefiles ging es noch) nicht mehr bauen lässt, da ich ja auf eine vorkompilierte libskindesignerapi lib verweise und beim Paketbau natürlich das libskindesignerapi Verzeichnis löschen muss.

Code

+ make 'CFLAGS=-O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches  -m64 -mtune=generic -fPIC' 'CXXFLAGS=-O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches  -m64 -mtune=generic -fPIC' -j3 all
make -C libskindesignerapi
make[1]: *** libskindesignerapi: No such file or directory.  Stop.
Makefile:119: recipe for target 'libskindesignerapi' failed

chriszero · April 6, 2015 at 7:54 PM

Das Plex Plugin wird auch ohne Skindesigner laufen, mit den Standard Listenmenüs eben. Und falls vorhanden wird eben das jeweilige Skin verwendet.

Copperhead · April 6, 2015 at 8:09 PM

Quote from marco

Weil einige Distributionen eine Politik haben, daß gebündelte Bibliotheken nicht mit ausgelierfert werden dürfen.
Siehe auch Fedora unbundle Politik und
Debian/Ubuntu unbundle Politik

So, und jetzt stehe ich dumm da. Ich finde nichts dergleichen bei Arch Linux.
Ich zerbreche mir schon eine halbe Stunde den Kopf, wo ich das her hab.

3PO · April 6, 2015 at 8:11 PM

Gentoo: --> https://wiki.gentoo.org/wiki/Why_not_bundle_dependencies

Participate now!