Re: [Mesa-dev] [PATCH v2 25/25] configure.ac: do not require the i965 driver for ANV

[Jason Ekstrand]

You left brw_defines.h in compiler/Makefile.sources.  With that fixed,

Reviewed-by: Jason Ekstrand 

On Thu, Mar 9, 2017 at 11:07 AM, Emil Velikov 
wrote:

> From: Emil Velikov 
>
> As of last commit we have the two split, thus we no longer require the
> i965 in order to have the ANV driver.
>
> Even though ANV does not link against libdrm nor libdrm_intel, we still
> require those as dependencies due to the headers they provide.
>
> Signed-off-by: Emil Velikov 
> ---
>  configure.ac | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 628aec3ceb..aa5cb75dfb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1786,9 +1786,8 @@ if test -n "$with_vulkan_drivers"; then
>  for driver in $VULKAN_DRIVERS; do
>  case "x$driver" in
>  xintel)
> -if test "x$HAVE_I965_DRI" != xyes; then
> -AC_MSG_ERROR([Intel Vulkan driver requires the i965 dri
> driver])
> -fi
> +require_libdrm "i965"
> +PKG_CHECK_MODULES([INTEL], [libdrm >= $LIBDRM_INTEL_REQUIRED
> libdrm_intel >= $LIBDRM_INTEL_REQUIRED])
>  HAVE_INTEL_VULKAN=yes
>
>  ;;
> --
> 2.11.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH v2 25/25] configure.ac: do not require the i965 driver for ANV

[Emil Velikov]

From: Emil Velikov 

As of last commit we have the two split, thus we no longer require the
i965 in order to have the ANV driver.

Even though ANV does not link against libdrm nor libdrm_intel, we still
require those as dependencies due to the headers they provide.

Signed-off-by: Emil Velikov 
---
 configure.ac | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index 628aec3ceb..aa5cb75dfb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1786,9 +1786,8 @@ if test -n "$with_vulkan_drivers"; then
 for driver in $VULKAN_DRIVERS; do
 case "x$driver" in
 xintel)
-if test "x$HAVE_I965_DRI" != xyes; then
-AC_MSG_ERROR([Intel Vulkan driver requires the i965 dri 
driver])
-fi
+require_libdrm "i965"
+PKG_CHECK_MODULES([INTEL], [libdrm >= $LIBDRM_INTEL_REQUIRED 
libdrm_intel >= $LIBDRM_INTEL_REQUIRED])
 HAVE_INTEL_VULKAN=yes
 
 ;;
-- 
2.11.1

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