Mesa (master): configure.ac: Ensure libomxil-bellagio exists before invoking pkg-config.

Matt Turner mattst88 at kemper.freedesktop.org
Tue Mar 7 15:27:56 UTC 2017


Module: Mesa
Branch: master
Commit: 69063d05612439ff8e60deda5b9847464d5b5873
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=69063d05612439ff8e60deda5b9847464d5b5873

Author: Matt Turner <mattst88 at gmail.com>
Date:   Mon Mar  6 10:49:05 2017 -0800

configure.ac: Ensure libomxil-bellagio exists before invoking pkg-config.

I was already tired of seeing the message

    Package libomxil-bellagio was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libomxil-bellagio.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libomxil-bellagio' found

on every configure, but I just got a distro bug reported where the user
was confused by this message and thought it indicated a bug.

Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

---

 configure.ac | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index bc85d3c..87cfbaf 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2195,7 +2195,8 @@ AC_ARG_WITH([omx-libdir],
     [AS_HELP_STRING([--with-omx-libdir=DIR],
         [directory for the OMX libraries])],
     [OMX_LIB_INSTALL_DIR="$withval"],
-    [OMX_LIB_INSTALL_DIR=`$PKG_CONFIG --define-variable=libdir=\$libdir --variable=pluginsdir libomxil-bellagio`])
+    [OMX_LIB_INSTALL_DIR=`$PKG_CONFIG --exists libomxil-bellagio && \
+                          $PKG_CONFIG --define-variable=libdir=\$libdir --variable=pluginsdir libomxil-bellagio`])
 AC_SUBST([OMX_LIB_INSTALL_DIR])
 
 dnl Directory for VA libs




More information about the mesa-commit mailing list