[Mesa-dev] [PATCH] Use AC_PATH_TOOL instead of AC_PATH_PROG for llvm-config.

[Tom Stellard]

On Sat, Dec 28, 2013 at 03:22:09PM +0100, Michał Górny wrote:
> This should help with cross-compiling and multilib when $CHOST-specific
> llvm-config is expected rather than build host default one.
> 
> It will help us a bit in Gentoo where we've started using
> i686-pc-linux-gnu-llvm-config for 32-bit multilib LLVM.
> 
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>

Should we CC stable on this patch?  Do you have commit access?

-Tom
> Signed-off-by: Michał Górny <mgorny at gentoo.org>
> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=73100
> ---
>  configure.ac | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index f75325d..1d68547 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1567,9 +1567,9 @@ if test "x$enable_gallium_llvm" = xauto; then
>  fi
>  if test "x$enable_gallium_llvm" = xyes; then
>      if test "x$llvm_prefix" != x; then
> -        AC_PATH_PROG([LLVM_CONFIG], [llvm-config], [no], ["$llvm_prefix/bin"])
> +        AC_PATH_TOOL([LLVM_CONFIG], [llvm-config], [no], ["$llvm_prefix/bin"])
>      else
> -        AC_PATH_PROG([LLVM_CONFIG], [llvm-config], [no])
> +        AC_PATH_TOOL([LLVM_CONFIG], [llvm-config], [no])
>      fi
>  
>      if test "x$LLVM_CONFIG" != xno; then
> -- 
> 1.8.5.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev