[wino]

D.h. wir haben wieder sauberes Build system für VDR? Greetz MarMic

Das denke ich nicht. Die Plugins mit neuem Makefile werden falsch gebaut und deshalb die Segfaults bei VDR. Ohne Make.config fehlen die Build Optionen der Patche.  

Gruß
wino

[wino]

bei der neuen Makefilelogik (auch nur Teilauszug!!!)

Code: [Select]

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/home/mld/MLD_uebe/dvb -c -DPLUGIN_NAME_I18N='"rssreader"' -I/home/mld/MLD_uebe/vdr/src/vdr-1.7.38/include  -o config.o config.c

Deshalb passen die Plugins mit neuem Makefile nicht zu der VDR Version. Die Folgeerscheinungen sind die Segfaults bei Benutzung dieser Plugins. Folglich muss das Buildsystem an den neuen Gegebenheiten angepasst werden.

Gruß
wino

[Christian]

Das sollte möglich sein

Einfach zu jedem Makefile ein Patch pro Addon

Oder

Wir lösen das global über die make Zeile und geben die Ensprechenden Parameter mit.

Dazu sollte Claus sich aber bitte mal melden

[clausmuus]

Ich glaube das muss ich mir mal ganz in Ruhe zu Gemüte führen. So wie ich das sehe, wird bei den Plugins nur noch die vdr.pc eingebunden, aber nicht mehr die Makefile.config. Das muss ich aber erst noch überprüfen.
Die vdr.pc wird beim kompilieren des VDRs erstellt. Ob die nun auf der Makefile.config basiert weiß ich noch nicht. Auf jeden Fall enthält die nicht alles was im Makefile.config definiert ist, sondern nur bestimmte Variablen.

Claus

[MarMic]

@claus,

so denke ich ist das auch. man muss sich diese vdr.pc an seine bedürfnisse anpassen, oder?

greetz MarMic

edit://
für was ist dann die plugins.conf?

Code: [Select]

>  # Use this if you want to have a central place where you configure compile time
>  # parameters for plugins:
> -#PLGCFG = $(CONFDIR)/plugins.conf
> +#PLGCFG = $(CONFDIR)/plugins.mk
sorry aber ich habe von dem ganzen make zeugs keinen plan


edit2://

grade die letzten beitraege im anounce thread:

Da hab ich nur DVBDIR und VIDEODIR gesetzt. Ist plgcfg für irgend etwas notwendig? Laufen tuts auch so.


Wird nur gebraucht wenn jemand (aus welchem Grund auch immer) eventuelle Parameter für Plugin-Makefiles global an zentraler Stelle pflegen will. In diesem Fall würdest du PLGCFG auf den Pfad zu dieser Datei mit Parametern setzen.

[clausmuus]

Hi,

So hab ich das auch verstanden. Brauchen wir also nicht.

Claus

[clausmuus]

@Pit,

die ganzen  -DUSE_... Optionen werden nicht mehr benötigt, seitdem wir den extpng Patch nicht mehr einsetzen.

Claus

[Christian]

Hab ich das jetzt richtig verstanden?

Die vdr.pc wird generiert aus dem vdr Build? Diese wird gefüllt aus der Make.config?
Muss aber aktiviert werden in der Make.config das die vdr.pc benutzt wird?

Grrrrr : soviel neues  :wand  

[clausmuus]

Hi,

hier mal meine Zusammenfassung, so wie ich die Neuerungen verstanden habe:

Beim make des VDRs wird eine vdr.pc Config Datei erstellt. Die enthält alle relevanten Daten zu Verzeichnissen und Compiler Einstellungen. Diese Daten werden aus den Vorgaben des vdr Makefile und dem Make.config erstellt. Da die meisten Vorgaben bereits passen, habe ich die Make.config mal auf des wirklich nötige reduziert. Die könnte prinzipiell auch ganz wegfallen, wenn ich noch drei Kleinigkeiten an anderer Stelle korrigiert habe.
Neue Plugin Makefiles benutzen den Befehl pkg-config um in der vdr.pc nach der benötigten Konfiguration zu suchen. Alte Makefiles bekommen Ihre Konfiguration aus den Umgebungs Variablen. Bei einem neuen Plugin Makefile werden z.B. die CFLAGS nun so gesetzt: CFLAGS = $(call PKGCFG,cflags)
Die neuen Plugin Makefiles haben nun den Vorteil, dass die auch außerhalb des VDR trees funktionieren sollten (noch nicht getestet). Dafür braucht beim make dann nur noch der Pfad zum VDR Verzeichnis (VDRDIR) angegeben zu werden. Das kommt allen Paketbauern (also auch uns) zu gute. Ich werde unser Plugin Make System also demnächst mal so umstellen, dass Plugins mit neuen Makefiles außerhalb des VDR trees gebaut werden (das erspart einiges an komplizierten hin und her kopiere).

Claus

[MarMic]

DANKE CLAUS!!!

GUT DAS ES DICH GIBT :-)

schau mal in den aktuellen anounce thread ich glaube da geht es um solche sachen

[wino]

Ich werde unser Plugin Make System also demnächst mal so umstellen, dass Plugins mit neuen Makefiles außerhalb des VDR trees gebaut werden (das erspart einiges an komplizierten hin und her kopiere).

Dann muss die jeweilige VDR Version im Devsystem installiert sein, bevor die Plugins gebaut werden.

Gruß
wino

[Christian]

Ich werde unser Plugin Make System also demnächst mal so umstellen, dass Plugins mit neuen Makefiles außerhalb des VDR trees gebaut werden (das erspart einiges an komplizierten hin und her kopiere).

Dann muss die jeweilige VDR Version im Devsystem installiert sein, bevor die Plugins gebaut werden.

Gruß
wino

Aber doch nur wenn wir VDRDIR nicht mit angeben, oder?

[clausmuus]

Hi,

Plugins lassen sich (und das wird auch so bleiben) nur dann bauen, wenn der VDR bereits gebaut wurde. Das wird im Addon Makefile gecheckt und gegebenenfalls nachgeholt. Ob's sich um nen neues oder altes Makefile handelt, werde ich genau so wie der VDR es tut prüfen.

Und ja, das geht, indem im src Verzeichnis des Plugins ein make unter Angabe des VDRDIR aufgerufen wird.

Claus

[MarMic]

das vdrdir wird doch so oder so in der vdr.pc enthalten sein...

ach ich glaub ich halt mich raus das ist mehr als rocket sience :-P

greetz MarMic

[P3f]

@Pit,

die ganzen  -DUSE_... Optionen werden nicht mehr benötigt, seitdem wir den extpng Patch nicht mehr einsetzen.

Claus

Hi Claus,

ich denke, auch wenn wir den Ext-Patch nicht anwenden(weil es ihn nicht mehr gibt), so sind bereits bei den Plugin Entwickler Abfragen dafür eingebaut. Das kann man sicherlich auch anders abfragen, aber wann werden wir alle Stellen finden.

Als Beispiel fällt mir da spontan "Extrecmenu" ein, dort gibt es mehrere Abfragen für den Graphtft-Patch.

Ansonsten teile ich deine Meinung vollends.

Gruß,

Pit