[vlc-devel] [PATCH 4/4] seek config.h include via angled bracket search

Marvin Scholz epirat07 at gmail.com
Sun Oct 4 22:01:21 CEST 2020


This looks like the patch I already proposed a while ago
and we basically did not do it because it would touch so many files
for very little gain.

On 2 Oct 2020, at 14:03, Steve Lhomme wrote:

> Here I think it's even more dangerous because if some contrib install 
> a config.h in their (direct) include path we may use that one rather 
> than the local one.

I am not sure this can actually happen (and it would be the exact same 
issue if not using angular includes)
as the path to the config.h folder will always come before other include 
flags, so it will search
those first.

Using angular includes is actually the recommended way according to the 
autotools documentation.


More information about the vlc-devel mailing list