[vlc-devel] [PATCH] configure: fail on casting incompatible pointers

Denis Charmet typx at dinauz.org
Wed Oct 2 14:55:57 CEST 2019


On 2019-10-02 14:43, Steve Lhomme wrote:
> On 2019-10-02 14:37, Denis Charmet wrote:
>> I agree with RĂ©mi here, using it on a small code base is good, using 
>> it in something as huge as VLC is more a hassle than anything else 
>> especially when you work on embedded platform with crazy broken 
>> toolchains.
> 
> Except we do not have any contribs with this issue, as I said I've
> been using it for month and it helped a lot.

I'm talking about other people building it with stupid toolchains not 
devs on their up to date computer.


>> And in the end you still have the warning to spot the errors.
> 
> Just like the commit that was pushed yesterday with a warning.
> 
> Warnings are precisely useless because we have a huge codebase. When
> building VLC there's way too much noise to actually see the good ones.

Then enforce it on CI not on the general configure.ac. It's good when 
you control all the build parameters but it's not the case for VLC. With 
this you end up with people just not using VLC because they can't be 
bothered to remove flags in autotools.

Regards,
-- 
Denis Charmet - TypX
Le mauvais esprit est un art de vivre


More information about the vlc-devel mailing list