Re: [PATCH v3] Gather necessary options for build in Make.global and include it.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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. [1][2][3]
>>>
>>> Therefore include strictly necessary options in `Make.global` and include this in all the Makefiles before `Make.config`.
>>>
>>> [1] http://www.linuxtv.org/pipermail/vdr/2009-July/020977.html
>>> [2] http://www.linuxtv.org/pipermail/vdr/2009-December/021807.html
>>> [3] 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. [4]
>>> v3: Change according to Frank’s comments [5].
>>>  • 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`.
>>>
>>> [4] http://www.gnu.org/software/make/manual/make.html#Include
>>> [5] http://www.linuxtv.org/pipermail/vdr/2010-January/022243.html
>>>
>>> Signed-off-by: Paul Menzel <paulepanter@xxxxxxxxxxxxxxxxxxxxx>
>>> ...
>> 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.

Klaus


_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux