On 23 February 2017 at 18:43, Leo Liu <leo....@amd.com> wrote: > Otherwise the configuration fails when building independant libs > like vdpau, vaapi or omx > I really should have spotted this usecase :-]
> Signed-off-by: Leo Liu <leo....@amd.com> > --- > configure.ac | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index 44c7883..890a379 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -2310,7 +2310,9 @@ if test -n "$with_gallium_drivers"; then > PKG_CHECK_MODULES([AMDGPU], [libdrm >= $LIBDRM_AMDGPU_REQUIRED > libdrm_amdgpu >= $LIBDRM_AMDGPU_REQUIRED]) > require_libdrm "radeonsi" > radeon_llvm_check $LLVM_REQUIRED_RADEONSI "radeonsi" > - require_basic_egl "radeonsi" > + if test "x$enable_egl" = xyes; then > + require_basic_egl "radeonsi" > + fi I'm split if we want to keep the check as-is or move within require_basic_egl. In either case: Fixes: 1ac40173c2a ("configure.ac: simplify EGL requirements for drivers dependent on EGL") Reviewed-by: Emil Velikov <emil.veli...@collabora.com> -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev