[vlc-devel] commit: Revert "Turn on -Werror for libvlccore and libvlc." ( Rémi Denis-Courmont )

Pierre d'Herbemont pdherbemont at free.fr
Thu Aug 14 03:11:01 CEST 2008


On Aug 14, 2008, at 2:48 AM, Mirsal Ennaime wrote:

> On Thu, Aug 14, 2008 at 1:32 AM, Pierre d'Herbemont <pdherbemont at free.fr 
> > wrote:
>> Ok. Do you agree then to turn -Werror when building when --enable- 
>> debug?
>
> I always build with --enable-debug as I use gdb quite extensively, but
> with my SoC project is hard enough not to give me any extra-time to
> fix warnings.

Well, you are not supposed to introduce warnings while coding :)  
That's one of the reason why I want to force usage of -Werror by  
default. Careless developer would have to be forced to fix their  
warnings.

>> An other idea is to --enable-werror on buildbot.
>
> Won't this bring problems on platforms that aren't used by the
> majority of developers ?

No. Breaking the buildbot is not a problem as we can easily fix them  
afterwards. Anyway the plan would be to enable -Werror on linux/osx  
(ie: Where it works without too much troubles).

Pierre.




More information about the vlc-devel mailing list