[vdr] [PATCH v3] Gather necessary options for build in Make.global and include it.
Klaus.Schmidinger at tvdr.de
Sat Feb 6 13:00:36 CET 2010
On 06.02.2010 12:43, Paul Menzel wrote:
> Am Samstag, den 06.02.2010, 12:29 +0100 schrieb Klaus Schmidinger:
>> On 29.01.2010 15:19, Paul Menzel wrote:
>>> Without this patch, if some options in `Makefile` were set outside `Makefile` and no `Make.config` existed with the necessary options, builds could fail. 
>>> Therefore include strictly necessary options in `Make.global` and include this in all the Makefiles before `Make.config`.
>>>  http://www.linuxtv.org/pipermail/vdr/2009-July/020977.html
>>>  http://www.linuxtv.org/pipermail/vdr/2009-December/021807.html
>>>  http://www.linuxtv.org/pipermail/vdr/2010-January/022235.html
>>> v2: Use `include` instead of `-include` to get a warning/error if it is not present. 
>>> v3: Change according to Frank’s comments .
>>> • Delete options set in `Make.global` from other Makefiles.
>>> • Leave adding `-fPIC` to `C[XX]FLAGS` in `Make.config.template` as the flags get reset including `Make.config`.
>>> • Modify `newplugin`.
>>>  http://www.gnu.org/software/make/manual/make.html#Include
>>>  http://www.linuxtv.org/pipermail/vdr/2010-January/022243.html
>>> Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
>> Is it safe to assume that this is the final version everybody
>> has agreed upon?
> I think so, since nobody commented on this anymore, it can be applied.
> Frank told me I should submit it to you. Great that you were faster.
> I do not know when you will release the 1.7.13. If it is still some days
> away then maybe we can set a limit of seven days to give everyone a
> chance again to comment on this patch and you can apply it afterward.
I'm either applying it today, or ignore it altogether ;-)
I believe there has been enough time for complaints.
More information about the vdr