[Mesa-dev] [PATCH] Revert "build: llvm libs may not be in system search path, add rpath"

Tom Stellard tom at stellard.net
Wed Mar 19 09:41:16 PDT 2014


On Mon, Mar 17, 2014 at 09:43:42AM -0400, Ilia Mirkin wrote:
> This reverts commit d9b983519c63b9072677364a6e399d213a1855e5.
> 
> Unfortunately it seems like rpath is evaluated before LD_LIBRARY_PATH,
> so this breaks e.g. stream, as well as any other user of that env var,
> if the llvm path happens to be where other libs also reside.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76082
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>  configure.ac | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 3f75266..2ebed46 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1592,7 +1592,6 @@ if test "x$enable_gallium_llvm" = xyes; then
>          LLVM_CXXFLAGS=`strip_unwanted_llvm_flags "$LLVM_CONFIG --cxxflags"`
>          LLVM_INCLUDEDIR=`$LLVM_CONFIG --includedir`
>          LLVM_LIBDIR=`$LLVM_CONFIG --libdir`
> -        LLVM_LDFLAGS="-Wl,-rpath,$LLVM_LIBDIR $LLVM_LDFLAGS"
>  
>          AC_COMPUTE_INT([LLVM_VERSION_MAJOR], [LLVM_VERSION_MAJOR],
>              [#include "${LLVM_INCLUDEDIR}/llvm/Config/llvm-config.h"])
> -- 
> 1.8.3.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list