commit 1914c814a6c4aaaf03c26f6c20a9871ebc98895f
author Emil Velikov <emil.velikov@collabora.com> Wed Dec 07 14:38:10 2016 +0000
committer Emil Velikov <emil.l.velikov@gmail.com> Fri May 19 19:46:54 2017 +0100
tree 87d7bb56f2f9f3796b8347958c01b3d79b7c8ad1
parent 63e11ac2b5c99fa351be5f61a035f72384609a1c

configure: error out if building OMX w/o supported platform

Cc: <mesa-stable@lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>

configure.ac

diff --git a/configure.ac b/configure.ac
index 90b70a6..e713a96 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2062,6 +2062,12 @@
 fi
 AM_CONDITIONAL(HAVE_ST_XA, test "x$enable_xa" = xyes)
 
+if echo $platforms | grep -q "x11\|drm"; then
+    have_omx_platform=yes
+else
+    have_omx_platform=no
+fi
+
 if echo $platforms | grep -q "x11\|drm\|wayland"; then
     have_va_platform=yes
 else
@@ -2080,7 +2086,7 @@
 	PKG_CHECK_EXISTS([vdpau >= $VDPAU_REQUIRED], [enable_vdpau=yes], [enable_vdpau=no])
     fi
 
-    if test "x$enable_omx" = xauto; then
+    if test "x$enable_omx" = xauto -a "x$have_omx_platform" = xyes; then
 	PKG_CHECK_EXISTS([libomxil-bellagio >= $LIBOMXIL_BELLAGIO_REQUIRED], [enable_omx=yes], [enable_omx=no])
     fi
 
@@ -2121,6 +2127,9 @@
 AM_CONDITIONAL(HAVE_ST_VDPAU, test "x$enable_vdpau" = xyes)
 
 if test "x$enable_omx" = xyes; then
+    if test "x$have_omx_platform" != xyes; then
+        AC_MSG_ERROR([OMX requires at least one of the x11 or drm platforms])
+    fi
     PKG_CHECK_MODULES([OMX], [libomxil-bellagio >= $LIBOMXIL_BELLAGIO_REQUIRED])
     gallium_st="$gallium_st omx"
 fi