[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