[vdr] [PATCH v2] Gather necessary options for build in Make.global and include it.

Paul Menzel paulepanter at users.sourceforge.net
Fri Jan 29 14:51:41 CET 2010


Dear Frank,


Am Freitag, den 29.01.2010, 10:04 +0100 schrieb Frank Schmirler:

[…]

> 1) from VDR's Makefile, remove the line
> DEFINES += -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
> It is already included from Make.global
> 
> 2) in all plugin Makefiles, remove -fPIC from the line
> CXXFLAGS ?= -fPIC -g -O2 -Wall -Woverloaded-virtual -Wno-parentheses
> It will be added by Make.global anyway

My reasoning was that someone reading the Makefile would have it easier
to see what options are needed. I will change it now.

I will also remove the line `DEFINES += -D_FILE …` in there too because
it should be added by `Make.global` too.

> 3) in Make.config.template, remove only the line
> DEFINES += -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE
> The lines with "+= -fPIC" are still necessary, as Make.config resets
> CFLAGS/CXXFLAGS.

Correct.

> 4) Script newplugin needs to be modified, too.

I did not know about `newplugin`.


Thanks for the review. I will send a new version of the patch,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Dies ist ein digital signierter Nachrichtenteil
URL: <http://www.linuxtv.org/pipermail/vdr/attachments/20100129/1d19b03d/attachment.pgp>


More information about the vdr mailing list