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

Jason Ekstrand jason at jlekstrand.net
Thu Mar 9 22:50:03 UTC 2017


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

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Thu, Mar 9, 2017 at 11:07 AM, Emil Velikov <emil.l.velikov at gmail.com>
wrote:

> From: Emil Velikov <emil.velikov at collabora.com>
>
> 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 <emil.velikov at collabora.com>
> ---
>  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 at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170309/b53b9d9f/attachment.html>


More information about the mesa-dev mailing list