[vlc-devel] [vlc-commits] configure.ac: Use AC_PATH_PROGS for luac
remi at remlab.net
Fri Sep 30 00:26:50 CEST 2016
Le jeudi 29 septembre 2016, 08:35:55 Steve Lhomme a écrit :
> > But that's the point. Don't touch the environment in configure (e.g. PATH)
> > - unless that change is exposed to the Makefile (e.g. CFLAGS). Otherwise,
> > the results of the configure tests are potentially invalidated.
> But that's exactly what we do in configure.ac, mess with the PATH to
> find the extra tools.
Yes and I think that's a bad idea. But I don't use extra/tools so I don't care
to fix, especially not if it means an argument with Rafaël, MacOS devs, or
> It's just not propagated after the configure
> call. So we end up discovering stuff but when running it, it doesn't
> We should be consistent one way or another.
Repeating yet again, but the only way to be consistent is to use
AC_CHECK_PROG(S), and keep off AC_PATH_PROG, since the first one is already used
More information about the vlc-devel