[Announce] VDR developer version 1.7.35

Zitat von Copperhead

kls:

Code

PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))

Das sollte gedreht werden. Das erst als letzter Ausweg im System nachgeschaut wird. Damit ein Plugin, dass innerhalb des VDR-Source liegt auch sicher gegen diesen gebaut wird und nicht gegen evtl. alte Header Dateien im System.

Code

PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) ../../../vdr.pc || pkg-config --variable=$(1) vdr))

Edit: Wenn ichs mir recht überlege kann man sich dann das ganze VDRDIR Zeug sparen.

Code

PKGCFG = $(shell pkg-config --variable=$(1) ../../../vdr.pc || pkg-config --variable=$(1) vdr))

Hat jemand eine bessere Idee, wie man aus dem Pluginverzeichnis raus erkennen könnte, dass man im VDR-Source liegt?

Alles anzeigen

VDRDIR ist imho ok. Ich würde allerdings das "../../../vdr.pc" rauswerfen. Denn entweder greift man auf $(VDRDIR)/vdr.pc oder das System-vdr.pc zu. Mehr braucht's nicht.

CU
Oliver

Zitat von Copperhead

kls:

Code

PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) vdr || pkg-config --variable=$(1) ../../../vdr.pc))

Das sollte gedreht werden. Das erst als letzter Ausweg im System nachgeschaut wird. Damit ein Plugin, dass innerhalb des VDR-Source liegt auch sicher gegen diesen gebaut wird und nicht gegen evtl. alte Header Dateien im System.

Code

PKGCFG = $(if $(VDRDIR),$(shell pkg-config --variable=$(1) $(VDRDIR)/vdr.pc),$(shell pkg-config --variable=$(1) ../../../vdr.pc || pkg-config --variable=$(1) vdr))

Edit: Wenn ichs mir recht überlege kann man sich dann das ganze VDRDIR Zeug sparen.

Code

PKGCFG = $(shell pkg-config --variable=$(1) ../../../vdr.pc || pkg-config --variable=$(1) vdr))

Hat jemand eine bessere Idee, wie man aus dem Pluginverzeichnis raus erkennen könnte, dass man im VDR-Source liegt?

Alles anzeigen

Wenn du "einfach so" ../../../vdr.pc nimmst, dann könnte das auch reiner Zufall sein, daß sich dort eine solche Datei befindet.
In der jetzigen Variante kann man über VDRDIR genau festlegen, wo die VDR-Source ist (die muß nicht zwingend in ../../.. sein).
Ist kein VDRDIR gesetzt, dann wird ein installiertes vdr.pc genommen, und wenn es auch kein solches gibt, dann wird als letzter Versuch ../../../vdr.pc genommen. Ich bin also nach wie vor der Meinung, daß es so, wie es jetzt ist, richtig ist.

Klaus

Zitat von UFO

VDRDIR ist imho ok. Ich würde allerdings das "../../../vdr.pc" rauswerfen. Denn entweder greift man auf $(VDRDIR)/vdr.pc oder das System-vdr.pc zu. Mehr braucht's nicht.

Das ist nötig für den Fall, daß man ein Plugin in seinem Source-Directory bauen will.

Klaus

Zitat von kls

Das ist nötig für den Fall, daß man ein Plugin in seinem Source-Directory bauen will.

Ok, hast recht.

CU
Oliver

Klar könnte zufälligerweise in ../../../ eine vdr.pc Datei liegen.

Mir ging es da eigentlich drum wenn ich in VDR-Source/PLUGINS/lib/hello make ausführe, zuerst die vdr.pc Datei aus dem VDR-Source genommen wird.

Weil VDRDIR ist in diesem Fall nicht gesetzt, nächster Schritt wäre dann, dass die vdr.pc aus dem System genommen wird, diese könnte aber von einer alten VDR-Version sein.

Zitat von Copperhead

Klar könnte zufälligerweise in ../../../ eine vdr.pc Datei liegen.

Mir ging es da eigentlich drum wenn ich in VDR-Source/PLUGINS/lib/hello make ausführe, zuerst die vdr.pc Datei aus dem VDR-Source genommen wird.

Weil VDRDIR ist in diesem Fall nicht gesetzt, nächster Schritt wäre dann, dass die vdr.pc aus dem System genommen wird, diese könnte aber von einer alten VDR-Version sein.

Wenn jemand einen VDR ins System installiert hat, dann muß er halt beim Bauen eines Plugins im Plugin-Source-Directory VDRDIR angeben, und sei's "../../..".
Auf jeden Fall ist man *mit* VDRDIR flexibler als ohne.

Klaus

Zitat von kls

Aus VDR-Sicht (lokaler Build):

- "make (all)" wie oben, aber zusätzlich werden die *.so-Dateien aller Plugins nach ./PLUGINS/lib und die *.mo-Dateien von VDR selber und der Plugins nach ./locale kopiert (hierfür ist es zwingend, daß die *.so Dateien im jeweiligen Source-Verzeichnis des Plugins entstehen, und die *.mo im jeweiligen "po"-Verzeichnis; wenn ein Plugin meint, sich nicht an diese Vorgabe halten zu müssen, dann funktioniert es eben nicht).

Machbar und vor allem auch ohne getrennten "Buildtype" lösbar. Allerdings hätte meine Variante (über "make install" gehen) den großen Vorteil gehabt, dass auch vom Plugin-Autoren vorgesehene Ergänzungen im "CONFDIR" und "RESDIR" nachgezogen worden wären. Ohne solche zusätzlichen Dateien funktionieren manche Plugins nicht und gerade hier sehe ich den sehr großen Vorteil der nun geschaffenen "make install"-Routine, dass die Plugin-Entwickler nun eine Stelle haben, an der solche Daten automatisch platziert werden können. Bisher musste man sich sowas in einer README oder INSTALL erst selber raussuchen und manuell platzieren.

Wenn dich das aber nicht stört, dann ist die von dir skizzierte Lösung in der Tat die simpelste Variante. Wenn Plugin-Autoren dann, weil ja im Makefile schon in Sourcecode-Form hinterlegt, in Zukunft auf separates Dokumentieren von "wo muss noch was hinkopiert werden" verzichten, dann müsste ein "in-place-Kompilierer" dann halt im Makefile nachlesen, was noch zu tun ist...

Moin!

Ich sehe, ihr kommt gut voran, ich wollte mal zwischendurch ein Lob aussprechen. Macht bitte weiter!

Prominentes, nicht-standard-konformes Plugin ist übrigens streamdev, falls ihr etwas "komisches" zum Testen haben wollt...
Mal abgesehen von den Unterverzeichnissen haben die Plugins "ungültige" Namen, da dort ein Minuszeichen auftaucht. Hat bisher aber auch nie gestört...

Lars.

Mir reichts für heute erstmal. Ich habe Klaus's "Skizze" fast umgesetzt. Ich schaffe es nur noch nicht die Locale an die richtige stelle zu kopieren.

Irgendwie nehmen die Plugins da ein Prozentzeichen, ich sehe aber irgendwie nicht, wo/wie das definiert wird.

Rom ist nicht an einem Tag erbaut worden... Nimm dir die Zeit, die du brauchst, bitte.

Lars.

Zitat von mini73

Rom ist nicht an einem Tag erbaut worden...

Genaugenommen ist Rom immer noch nicht fertig...

Das Hauptproblem ist das UP3-Makro. Folgende Variante funktioniert für mich. 'make' erzeugt auch alle Files in PLUGINS/lib und locale, unabhängig davon, wohin die Install-Verzeichnisse zeigen.

diff -u Makefile.save Makefile
--- Makefile.save       2012-12-30 12:18:18.000000000 +0100
+++ Makefile    2013-01-01 21:21:26.728192464 +0100
@@ -23,7 +23,7 @@


 # Directories:


-CWD      = .
+CWD      = $(shell pwd)
 LSIDIR   = ./libsi
 DESTDIR ?=
 PREFIX  ?= /usr/local
@@ -52,8 +52,6 @@
 CFLAGS += -I$(DVBDIR)
 endif


-UP3 = $(if $(findstring "$(LIBDIR)-$(LOCDIR)","$(CWD)/PLUGINS/lib-$(CWD)/locale"),../../../,)
-
 SILIB    = $(LSIDIR)/libsi.a


 OBJS = audio.o channels.o ci.o config.o cutter.o device.o diseqc.o dvbdevice.o dvbci.o\
@@ -111,6 +109,9 @@
 # Dependencies:


 MAKEDEP = $(CXX) -MM -MG
+ifdef DVBDIR
+MAKEDEP += -I$(DVBDIR)
+endif
 DEPFILE = .dependencies
 $(DEPFILE): Makefile
        @$(MAKEDEP) $(DEFINES) $(INCLUDES) $(OBJS:%.o=%.c) > $@
@@ -137,12 +138,12 @@
        @echo "videodir=$(VIDEODIR)" >> $@
        @echo "cachedir=$(CACHEDIRDEF)" >> $@
        @echo "resdir=$(RESDIRDEF)" >> $@
-       @echo "libdir=$(UP3)$(LIBDIR)" >> $@
-       @echo "locdir=$(UP3)$(LOCDIR)" >> $@
+       @echo "libdir=$(LIBDIR)" >> $@
+       @echo "locdir=$(LOCDIR)" >> $@
        @echo "plgcfg=$(PLGCFG)" >> $@
        @echo "apiversion=$(APIVERSION)" >> $@
-       @echo "cflags=$(CFLAGS) $(CDEFINES) -I$(UP3)$(INCDIR)" >> $@
-       @echo "cxxflags=$(CXXFLAGS) $(CDEFINES) -I$(UP3)$(INCDIR)" >> $@
+       @echo "cflags=$(CFLAGS) $(CDEFINES) -I$(INCDIR)" >> $@
+       @echo "cxxflags=$(CXXFLAGS) $(CDEFINES) -I$(INCDIR)" >> $@
        @echo "" >> $@
        @echo "Name: VDR" >> $@
        @echo "Description: Video Disk Recorder" >> $@
@@ -176,8 +177,10 @@
 i18n: $(I18Nmsgs) $(I18Npot)


 install-i18n:
-       @mkdir -p $(DESTDIR)$(LOCDIR)
-       @(cd $(LOCALEDIR); cp -r --parents * $(DESTDIR)$(LOCDIR))
+       @if [ "$(CWD)/$(LOCALEDIR)" != "$(LOCDIR)" ]; then\
+               mkdir -p $(DESTDIR)$(LOCDIR);\
+               cd $(LOCALEDIR); cp -r --parents * $(DESTDIR)$(LOCDIR);\
+       fi


 # The 'include' directory (for plugins):


@@ -208,17 +211,13 @@
               echo "* VDR version 1.7.35. Support for old style Makefiles may be dropped";\
               echo "* in future versions of VDR.";\
               echo "********************************************************************";\
-              $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" CXXFLAGS="$(CXXFLAGS)" VDRDIR=$(UP3) LIBDIR=../../lib all || failed="$$failed $$i";\
+              $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" CXXFLAGS="$(CXXFLAGS)" VDRDIR=$(CWD) LIBDIR=../../lib all || failed="$$failed $$i";\
            else\
-               target=all;\
-              if [ "$(LIBDIR)" = "$(CWD)/PLUGINS/lib" ] && [ "$(LOCDIR)" = "$(CWD)/locale" ]; then\
-                 target="install";\
-                 fi;\
               includes=;\
               if [ "$(INCDIR)" != "$(CWD)/include" ]; then\
-                 includes="INCLUDES=-I$(UP3)/include";\
+                 includes="INCLUDES=-I$(CWD)/include";\
                  fi;\
-              $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" VDRDIR=$(UP3) $$includes $$target || failed="$$failed $$i";\
+              $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" VDRDIR=$(CWD) LOCDIR=$(CWD)/locale LIBDIR=$(CWD)/PLUGINS/lib $$includes install || failed="$$failed $$i";\
               fi;\
            done;\
        if [ -n "$$noapiv" ] ; then echo; echo "*** plugins without APIVERSION:$$noapiv"; echo; fi;\
@@ -262,14 +261,25 @@


 install-plugins: plugins
        @for i in `ls $(PLUGINDIR)/src | grep -v '[^a-z0-9]'`; do\
-            $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" VDRDIR=$(UP3) DESTDIR=$(DESTDIR) install;\
-            done
+           newmakefile=`grep "PKGCFG" "$(PLUGINDIR)/src/$$i/Makefile"`;\
+           if [ -z "$$newmakefile" ]; then\
+               if [ "$(PLUGINDIR)/lib" != "$(LIBDIR)" ]; then\
+                   if [ -e $(PLUGINDIR)/lib/libvdr-$$i.so.$(APIVERSION) ]; then\
+                       cp --remove-destination $(PLUGINDIR)/lib/libvdr-$$i.so.$(APIVERSION) $(LIBDIR)/libvdr-$$i.so.$(APIVERSION);\
+                   fi;\
+               fi;\
+           else\
+               $(MAKE) --no-print-directory -C "$(PLUGINDIR)/src/$$i" VDRDIR=$(CWD) DESTDIR=$(DESTDIR) install;\
+           fi;\
+       done


 # Includes:


 install-includes: include-dir
-       @mkdir -p $(DESTDIR)$(INCDIR)
-       @cp -pLR include/vdr include/libsi $(DESTDIR)$(INCDIR)
+       @if [ "$(CWD)/include" != "$(DESTDIR)$(INCDIR)" ]; then\
+               mkdir -p $(DESTDIR)$(INCDIR);\
+               cp -pLR include/vdr include/libsi $(DESTDIR)$(INCDIR);\
+       fi


 # pkg-config file:

Gruß
e9hack

Zitat von 3PO

Genaugenommen ist Rom immer noch nicht fertig...

Schon richtig, aber länger als 6 Tage sollte es auch nicht dauern

Zitat

Und er sah, dass es gut war

Gerald

Zitat von 3PO

Das mag bei dem Debian Zeugs wohl so sein, bei Gentoo hat man die Sourcen immer (gepackt) auf der HDD.

Sprich nicht von Gentoo wenn Du den Kram von Hand installierst

Die von Klaus erdachten änderungen habe ich bereits fuer vdr-1.7.35 umgesetzt,
wo möglich, hab ich das "neumodische" Zeug von klaus dafuer benutzt.
Die install der Plugins functioniert mit alter und neuer Makefile structur.

Anyway, da da alles in einer Sandbox kompiliert wird, müssen/werden die notwendign Umgebungsvariablen je nach bedarf fuer das gentoo build system herangeholt, ggf. korrigiert.
Das funtioniert natürlich nur, wenn man per paketmanger installiert.

Selbstgepresst ist eigenverantwortung und hat nix mit gentoo zu tun
Dazu kann jede Distributione herhalten, soweit irgendein compiler installiert ist.

btw. Gesundes Neues @all

Zitat von Copperhead

Mir reichts für heute erstmal.

Laß dir Zeit, es drängt dich doch niemand!
Wie schon an anderer Stelle gesagt wurde, wenn es aufhört, Spaß zu machen, dann ist es Arbeit - und das soll dieses Hobby doch nicht werden

Zitat

Ich habe Klaus's "Skizze" fast umgesetzt. Ich schaffe es nur noch nicht die Locale an die richtige stelle zu kopieren.

Irgendwie nehmen die Plugins da ein Prozentzeichen, ich sehe aber irgendwie nicht, wo/wie das definiert wird.

Meinst du diese Zeile hier?

$(I18Nmsgs): $(LOCDIR)/%/LC_MESSAGES/vdr-$(PLUGIN).mo: $(PODIR)/%.mo

Hier bildet "$(PODIR)/%.mo" einen Pattern, und der '%'-Anteil davon wird jeweils in "$(LOCDIR)/%/LC_MESSAGES/vdr-$(PLUGIN).mo" eingesetzt um ein Target zu bilden. Auf diese Weise werden für alle *.mo-Dateien die vollständigen Pfadnamen im $(LOCDIR) ermittelt.

Klaus

Zitat von 3PO

Genaugenommen ist Rom immer noch nicht fertig...

Ich finde Rom ist ganz schoen fertig

Zitat von helau

Ich finde Rom ist ganz schoen fertig

Und ganz schön Pleite ...

SCNR

Hallo,

ich möchte einmal das Thema wechseln.
Mit vdr-1.7.35 (1.7.34 habe ich übersprungen) habe ich das Problem, dass HD-Aufnahmen nicht korrekt bis zum Ende abgespielt werden.
Ca. 20 Sekunden vor dem Aufnahmeende verfällt der vdr in eine extrem langsame Zeitlupe und reagiert auch nur noch schlecht auf die Eingaben der Fernbedienung.
Ton kommt nur sporadisch!
Wenn ich nicht mehrfach mit der STOP-Taste die Wiedergabe beende, braucht er ca. 2 min. um das Ende zu erreichen!
Mit meiner aktuellen Konfiguration (siehe Signatur) läuft alles Bestens. ** Hierfür mal ein großen DANKE an Klaus und alle anderen Entwickler! **
Hat sonst noch jemand das Problem?

Gruß
grappi