On Tue, Nov 01, 2016 at 01:42:17PM -0400, Lennart Sorensen wrote: > Well that looks like it only adds it for libvlccore.
So at least something like this is needed, but I think the VLCCORE is wrong too, and maybe the deinterlace has to be moved to only merge.c rather than all of deinterlace. --- configure.ac.orig 2016-11-03 13:56:25.826878272 -0400 +++ configure.ac 2016-11-03 13:56:55.558878272 -0400 @@ -1431,7 +1431,6 @@ ac_cv_c_altivec="no" ]) ]) - VLC_RESTORE_FLAGS AS_IF([test "${ac_cv_c_altivec}" != "no"], [ CPPFLAGS="${CPPFLAGS} ${ac_cv_c_altivec}" AC_DEFINE(CAN_COMPILE_C_ALTIVEC, 1, [Define to 1 if C AltiVec extensions are available.]) @@ -1441,6 +1440,7 @@ have_altivec="yes" ]) AC_CHECK_HEADERS(altivec.h) + VLC_RESTORE_FLAGS VLC_SAVE_FLAGS LDFLAGS="${LDFLAGS} -Wl,-framework,vecLib" The CPPFLAGS are being clobbered with the -maltivec, which is only needed when testing altivec.h, and should be removed again after that. That removes the -maltivec that was showing up everywhere at least, and now it is only in a few places, although perhaps still a few too many. -- Len Sorensen