Re: [Mesa-dev] [PATCH 3/3] configure.ac: report an error if LLVM shared libs are disabled and CL is enabled

2013-10-09 Thread Marek Olšák
You should use --with-llvm-shared-libs if you don't use it already.

Marek

On Wed, Oct 9, 2013 at 1:54 AM, Mike Lothian m...@fireburn.co.uk wrote:
 Just had a failure because of this - should the default for shared libs not
 be yes now?


 On 7 October 2013 13:27, Tom Stellard t...@stellard.net wrote:

 On Mon, Oct 07, 2013 at 01:05:15PM +0200, Marek Olšák wrote:
  From: Marek Olšák marek.ol...@amd.com
 

 Reviewed-by: Tom Stellard thomas.stell...@amd.com

  ---
   configure.ac | 5 +++--
   1 file changed, 3 insertions(+), 2 deletions(-)
 
  diff --git a/configure.ac b/configure.ac
  index 559c9a3..6158c6e 100644
  --- a/configure.ac
  +++ b/configure.ac
  @@ -1532,8 +1532,9 @@ AC_ARG_WITH([llvm-shared-libs],
   [with_llvm_shared_libs=no])
   AS_IF([test x$enable_opencl = xyes],
   [
  -AC_MSG_WARN([OpenCL required, forcing LLVM shared libraries])
  -with_llvm_shared_libs=yes
  +if test x$with_llvm_shared_libs != xyes; then
  +AC_MSG_ERROR([OpenCL requires LLVM shared libraries])
  +fi
   ])
 
   AC_ARG_WITH([llvm-prefix],
  --
  1.8.1.2
 
  ___
  mesa-dev mailing list
  mesa-dev@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/mesa-dev
 ___
 mesa-dev mailing list
 mesa-dev@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/mesa-dev


___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 3/3] configure.ac: report an error if LLVM shared libs are disabled and CL is enabled

2013-10-08 Thread Mike Lothian
Just had a failure because of this - should the default for shared libs not
be yes now?


On 7 October 2013 13:27, Tom Stellard t...@stellard.net wrote:

 On Mon, Oct 07, 2013 at 01:05:15PM +0200, Marek Olšák wrote:
  From: Marek Olšák marek.ol...@amd.com
 

 Reviewed-by: Tom Stellard thomas.stell...@amd.com

  ---
   configure.ac | 5 +++--
   1 file changed, 3 insertions(+), 2 deletions(-)
 
  diff --git a/configure.ac b/configure.ac
  index 559c9a3..6158c6e 100644
  --- a/configure.ac
  +++ b/configure.ac
  @@ -1532,8 +1532,9 @@ AC_ARG_WITH([llvm-shared-libs],
   [with_llvm_shared_libs=no])
   AS_IF([test x$enable_opencl = xyes],
   [
  -AC_MSG_WARN([OpenCL required, forcing LLVM shared libraries])
  -with_llvm_shared_libs=yes
  +if test x$with_llvm_shared_libs != xyes; then
  +AC_MSG_ERROR([OpenCL requires LLVM shared libraries])
  +fi
   ])
 
   AC_ARG_WITH([llvm-prefix],
  --
  1.8.1.2
 
  ___
  mesa-dev mailing list
  mesa-dev@lists.freedesktop.org
  http://lists.freedesktop.org/mailman/listinfo/mesa-dev
 ___
 mesa-dev mailing list
 mesa-dev@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/mesa-dev

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH 3/3] configure.ac: report an error if LLVM shared libs are disabled and CL is enabled

2013-10-07 Thread Marek Olšák
From: Marek Olšák marek.ol...@amd.com

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

diff --git a/configure.ac b/configure.ac
index 559c9a3..6158c6e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1532,8 +1532,9 @@ AC_ARG_WITH([llvm-shared-libs],
 [with_llvm_shared_libs=no])
 AS_IF([test x$enable_opencl = xyes],
 [
-AC_MSG_WARN([OpenCL required, forcing LLVM shared libraries])
-with_llvm_shared_libs=yes
+if test x$with_llvm_shared_libs != xyes; then
+AC_MSG_ERROR([OpenCL requires LLVM shared libraries])
+fi
 ])
 
 AC_ARG_WITH([llvm-prefix],
-- 
1.8.1.2

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 3/3] configure.ac: report an error if LLVM shared libs are disabled and CL is enabled

2013-10-07 Thread Tom Stellard
On Mon, Oct 07, 2013 at 01:05:15PM +0200, Marek Olšák wrote:
 From: Marek Olšák marek.ol...@amd.com
 

Reviewed-by: Tom Stellard thomas.stell...@amd.com

 ---
  configure.ac | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)
 
 diff --git a/configure.ac b/configure.ac
 index 559c9a3..6158c6e 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -1532,8 +1532,9 @@ AC_ARG_WITH([llvm-shared-libs],
  [with_llvm_shared_libs=no])
  AS_IF([test x$enable_opencl = xyes],
  [
 -AC_MSG_WARN([OpenCL required, forcing LLVM shared libraries])
 -with_llvm_shared_libs=yes
 +if test x$with_llvm_shared_libs != xyes; then
 +AC_MSG_ERROR([OpenCL requires LLVM shared libraries])
 +fi
  ])
  
  AC_ARG_WITH([llvm-prefix],
 -- 
 1.8.1.2
 
 ___
 mesa-dev mailing list
 mesa-dev@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/mesa-dev
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev