[Mesa-dev] [PATCH 1/1] configure: Bail out on llvm-config component error

[Nicolai Hähnle]

On 12.02.2016 19:41, Jan Vesely wrote:
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> ---
>   configure.ac | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 2750d4d..57330cb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2295,6 +2295,9 @@ dnl in LLVM_LIBS.
>
>   if test "x$MESA_LLVM" != x0; then
>
> +    if ! $LLVM_CONFIG --libs ${LLVM_COMPONENTS} >/dev/null; then
> +       AC_MSG_ERROR([Calling ${LLVM_CONFIG} failed])
> +    fi
>       LLVM_LIBS="`$LLVM_CONFIG --libs ${LLVM_COMPONENTS}`"
>
>       dnl llvm-config may not give the right answer when llvm is a built as a
>

+1 for making the build process more user-friendly.

Reviewed-by: Nicolai Hähnle <nicolai.haehnle at amd.com>