Neues Makfile und Paktetbau. Grundsätzliche Frage(n).

Zitat von PaulPanther

Wenn ich es richtig verstanden habe ließt das neue Makefile nur noch die vdr.pc wo leider ohne den Patch oben die Defines zu den Patchen fehlt ....

Jup, der Patch muss halt "#define USE_*" in einem passenden VDR Headerfile setzen, dann bekommt das Plugin es auch mit. Ferner ist es unnötig den mit "#ifdef USE_*" abschaltbar zu machen.

Es macht ja bei Einzelpatches keinen Sinn das in die Make.config zu packen, das war ja nur bei Multipatches so üblich.

BTW: Die Problematik wurde ja schon mal im "neues Makefile" Thread angesprochen und dann auch so geklärt.

cu

Genau ... Zur Zeit benutzen wir (mld) noch mit den extpng und dort hatten wir das Problem ... Ich hoffe nur das der Patch oben nicht noch andere Probleme macht da alle Defines gesetzt werden ....

Leider bin ich nicht so der Makefile Experte

Haste mal ein link?

Zitat von PaulPanther

Genau ... Zur Zeit benutzen wir (mld) noch mit den extpng

Ich hoffe, gerade da jetzt auch einige Patches überflüssig wurden, das die Patches zukünftig als Einzelpatches kommen. Dann fällt auch die Pflege der einzelnen Patches auch leichter.

Zitat von PaulPanther

Haste mal ein link?

Das war irgendwo in ner ewig langen Diskusion.

Aber im Prinzip muss ja nur (für nen Multipatch) das was in der Make.config ist da bleiben (damit die Patches im vdr Quellcode aktiviert sind) und in der PLGCFG (muss halt dann auch in das vdr-dev damit die Pluginmakefiles sie einbinden) kommen dann nochmal die
---
DEFINES += -DUSE_<patch a>
DEFINES += -DUSE_<patch b>
...
---
für alle (in der Make.config) aktivierten Patches. Dann bekommen die Plugins das über diesen Weg.

cu

Kannst Du das auch mal probieren?

Diesen Wert in der Make.config aktivieren und anpassen.

PLGCFG = $(VDRDIR)/Make.config

Zitat von Keine_Ahnung

Aber dann sind die Patches im VDR Paket fehlerhaft, weil sie müssten "#define USE_*" im VDR Quellcode setzen damit so was gar nicht erst vorkommt.

Also ich musste waehrend dieser ganzen Makefile-Odyssee schmerzlich lernen, dass diese Patch-Defines sehr wohl auch indirekt das Ergebnis des Bauens eines Plugins beeinflussen, welches die entsprechenden Patches gerade NICHT nutzt, bis hin zu Undefined Symbols beim Linken oder Segfaults zur Laufzeit. Der Hintergrund ist wohl der, dass manchmal diese Patches auch die Header (also die API) veraendern, und selbst wenn jene API-Funktionen vom Plugin nicht direkt aufgerufen werden, kommnt es oft vor dass sie trotzdem ueber mehrere Ecken an API-Aufrufen doch genutzt werden, also quasi versteckt sind. Also hoert meinen Rat, bei gepatchtem VDR, baut ALLE Plugins mit den ganzen Patch-Defines, wenn dann das Kompilieren erfolgreich geht, sollte der Rest in der Regel auch klappen...

Just my 2 cents, Lucian

btw: so reicht natürlich auch

--- 1/Makefile.org	2013-02-26 23:57:29.264790381 +0100
+++ 2/Makefile	2013-02-26 23:56:57.000000000 +0100
@@ -167,8 +167,8 @@
 	@echo "locdir=$(LOCDIR)" >> $@
 	@echo "plgcfg=$(PLGCFG)" >> $@
 	@echo "apiversion=$(APIVERSION)" >> $@
-	@echo "cflags=$(CFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
-	@echo "cxxflags=$(CXXFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
+	@echo "cflags=$(CFLAGS) $(DEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
+	@echo "cxxflags=$(CXXFLAGS) $(DEFINES) $(CINCLUDES) $(HDRDIR)" >> $@
 	@echo "" >> $@
 	@echo "Name: VDR" >> $@
 	@echo "Description: Video Disk Recorder" >> $@