[Announce] VDR developer version 1.7.35

Ja - aber ich glaube im Satz von glotzipapa fehlt ein "unterhalb"...
Wenn die LCLBLD definitionen nicht greifen, greifen die defaults aus Makefile, und da steht PREFIX auf /usr/local - LIBDIR wird im Makfile: LIBDIR ?= $(PREFIX)/lib/vdr.

Ich habe noch ein paar Kleinigkeiten. In Make.config greift

LIBDIR     ?= $(PLUGINDIR)/lib

niemals, weil es ja schon in Makefile belegt wurde. Ich habe mir hier so geholfen:

ifndef PLUGIN
LIBDIR     = $(PLUGINDIR)/lib
endif

Wenn also LCLBLD gesetzt ist, und es kein Plugin ist, dann greift es - dann wird auch u.a. eine vdr.pc mit den Werten aus Make.config erzeugt.
Außerdem gibst Du im Makefile zwar CINCLUDES (siehe auch die Überschreibe-Problematik mit = anstatt von += in meinem anderen Posting) ans vdr.pc weiter, aber nicht mehr im compiler-Aufruf für die targets. Ich habe es bei mir in die implizite Regel eingebaut:

%.o: %.c
        $(CXX) $(CXXFLAGS) -c $(DEFINES) $(INCLUDES) $(CINCLUDES) $<

Und damit die Compiler-Optionen für die Header vom Media-Build-Experimental auch in allen Compler-Aufrufen auftauchen (sonst übersetzt das nämlich nicht mit diesen Headern, habe ich in Make.config vor dem ifdev PLUGIN

DVBDIR   = /usr/local/src/media_build_experimental/linux/include/uapi
ifdef DVBDIR
CFLAGS  += -D__user= -D__u8=uint8_t
endif

Verstehe das jetzt nicht falsch, wenn Du das Bauen gegen die Treiber-Header für überflüssig hälst, dann lass es ganz raus, dann müssen die INCLUDES aber auch nicht in die CFLAGS bzw. CXXFLAGS des vdr.pc. Ganz oder gar nicht - ich kann mit beidem leben und mich selbst drum kümmern, ich schreibe es hir nur, damit sich keiner wundert, warum er so schöne -Iblafasel/uapi Zeilen in vdr.pc hat, aber es nirgendwo ankommt...

Gruß, Ingo

Ach, einen habe ich noch - der geht aber eher an Plugin-Entwickler: mir war skinnopacity (mit neuem 1.7.35 Makefile) auf die Nase gefallen, weil dort das INCLUDES += im Makefile nicht mehr anzog (wegen des INCLUDES hinter make im Aufruf aus dem vdr-Makefile). Im Pluginverzeichnis bauen ging - "von oben" bauen ging nicht. Ein "override" vor INCLUDES wird für die PLUGIN-makefiles also vrrpflichtend, wenn der Aufruf so bleibt.

Gruß, Ingo

Zitat von Copperhead

Da die INCLUDES in den Plugins mit += definiert werden geht es auch so.

...hätte ich bis gestern auch so gesehen. Vieleichtbhabe ich aber gerade einen Konoten im Hirn - ich sehe im Moment vor lauter ?=, +=, =, ifdef und ifndev den Wald nicht mehr... (ich glaube Du weisst noch viel besser, was ich meine...)

Zwei Sachen laufen bei v12 jedenfalls min. Noch in die Wand: im Makefile wird in CINCLUDES immer noch DVBDIR 4 Zeilen später durch ein fehlendes '+' überschrieben. Und die Compiler-Optionen "-D__user= -D__u8=uint8_t" fehlen. Sorry.

Gruß, Ingo

P.S.: Ich teste im Moment nur LCLBLD ohne make install mit DVBDIR.
P.P.S.: So groß, wie das Geschrei nach der Umstellung war, verstehe ich nicht, dass hier nicht viel mehr Leute versuch Copperhead zu unterstützen. Ich sehe es schon kommen: irgendwann kommt eine 1.7.36, und das große Heulen setzt wieder ein.
P.P.P.S.: Je länger ich mir diese Makefiles ansehe, um so besset finde ich die Idee mit vdr.pc und Patches z.B. als Versionsvermerk in irgendeine config.h zu schreiben, und um so dankbarer bin ich Chopperhead, dass er sich da ran wagt!

Zitat von nvertigo

...hätte ich bis gestern auch so gesehen. Vieleichtbhabe ich aber gerade einen Konoten im Hirn - ich sehe im Moment vor lauter ?=, +=, =, ifdef und ifndev den Wald nicht mehr...

6.7 The override Directive

If a variable has been set with a command argument (see Overriding Variables), then ordinary assignments in the makefile are ignored. If you want to set the variable in the makefile even though it was set with a command argument, you can use an override directive, which is a line that looks like this:

override variable = value

or

override variable := value

To append more text to a variable defined on the command line, use:

override variable += more text

See Appending More Text to Variables.

Variable assignments marked with the override flag have a higher priority than all other assignments, except another override. Subsequent assignments or appends to this variable which are not marked override will be ignored.

The override directive was not invented for escalation in the war between makefiles and command arguments. It was invented so you can alter and add to values that the user specifies with command arguments.

For example, suppose you always want the ‘-g’ switch when you run the C compiler, but you would like to allow the user to specify the other switches with a command argument just as usual. You could use this override directive:

override CFLAGS += -g

You can also use override directives with define directives. This is done as you might expect:

override define foo =
bar
endef

See Defining Multi-Line Variables. Quelle:GNU make

Und wie gesagt: habe ich mir gestern raus gesucht, als skinnopacity in die Wand lief, hätte sonst auch gedacht, das += ohne override anhängt. Hier lernt man fürs Leben...

Gruß, Ingo