[Mesa-dev] [PATCH] configure: set HAVE_COMMON_DRI when building only swrast

2013-10-03 Thread Emil Velikov
With commit cb1febb07, I have incorrectly removed HAVE_COMMON_DRI
assuming that swrast does not need to build the translations for
driconf options, as effectively swrast/drisw does not use them.

With the incoming unification work of dri and drisw, it makes
sense just to revert the offending hunk.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70057
Reported-by: Vinson Lee v...@freedesktop.org
Tested-by: Vinson Lee v...@freedesktop.org
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---

Vinson as I'm short of commit access (for obvious reasons) so feel
free to commit if you're happy with this (and noone objects).
I've made a silly assumption, which I'll try not to repeat in the
future.

Cheers
Emil
---

 configure.ac | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configure.ac b/configure.ac
index e7c8223..9546163 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1823,6 +1823,7 @@ if test x$with_gallium_drivers != x; then
 
 if test x$enable_dri = xyes; then
 GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS dri-swrast
+HAVE_COMMON_DRI=yes
 fi
 if test x$enable_vdpau = xyes; then
 GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS vdpau-softpipe
-- 
1.8.4

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


[Mesa-dev] [PATCH] configure: set HAVE_COMMON_DRI when building only swrast

2013-10-02 Thread Emil Velikov
With commit cb1febb07, I have incorrectly removed HAVE_COMMON_DRI
assuming that swrast does not need to build the the translations
for driconf options, as effectively swrast/drisw does not use them.

With the incomming unification work of dri and drisw, it makes
sense just to revert the offending hunk.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70057
Reported-by: Vinson Lee v...@freedesktop.org
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
 configure.ac | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configure.ac b/configure.ac
index e7c8223..9546163 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1823,6 +1823,7 @@ if test x$with_gallium_drivers != x; then
 
 if test x$enable_dri = xyes; then
 GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS dri-swrast
+HAVE_COMMON_DRI=yes
 fi
 if test x$enable_vdpau = xyes; then
 GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS vdpau-softpipe
-- 
1.8.4

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


Re: [Mesa-dev] [PATCH] configure: set HAVE_COMMON_DRI when building only swrast

2013-10-02 Thread Ian Romanick
On 10/02/2013 03:27 PM, Emil Velikov wrote:
 With commit cb1febb07, I have incorrectly removed HAVE_COMMON_DRI
 assuming that swrast does not need to build the the translations
  ^^^
Just the. :)

 for driconf options, as effectively swrast/drisw does not use them.
 
 With the incomming unification work of dri and drisw, it makes
   ^
incoming

 sense just to revert the offending hunk.
 
 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70057
 Reported-by: Vinson Lee v...@freedesktop.org
 Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
 ---
  configure.ac | 1 +
  1 file changed, 1 insertion(+)
 
 diff --git a/configure.ac b/configure.ac
 index e7c8223..9546163 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -1823,6 +1823,7 @@ if test x$with_gallium_drivers != x; then
  
  if test x$enable_dri = xyes; then
  GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS dri-swrast
 +HAVE_COMMON_DRI=yes
  fi
  if test x$enable_vdpau = xyes; then
  GALLIUM_TARGET_DIRS=$GALLIUM_TARGET_DIRS vdpau-softpipe
 

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