[Spice-devel,client,04/11] build-sys: Remove some dead configure.ac DBus code

Submitted by Francois Gouget on Nov. 3, 2015, 12:05 p.m.

Details

Message ID alpine.DEB.2.20.1511031251030.11833@amboise
State New
Headers show
Series "Series without cover letter" ( rev: 1 ) in Spice

Not browsing as part of any series.

Commit Message

Francois Gouget Nov. 3, 2015, 12:05 p.m.
Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
---
 configure.ac | 4 ----
 1 file changed, 4 deletions(-)

This patch does not depend on any other patch in this series.

Patch hide | download patch | download mbox

diff --git a/configure.ac b/configure.ac
index 5790a37..7033cbb 100644
--- a/configure.ac
+++ b/configure.ac
@@ -671,10 +671,6 @@  have_dbus=no
 if test "x$enable_dbus" != "xno"; then
   AC_DEFINE([USE_GDBUS], [1], [Define if supporting gdbus])
   have_dbus=yes
-
-  if test "x$enable_dbus" = "xyes" && test "x$have_dbus" = "xno"; then
-    AC_MSG_ERROR([D-Bus support explicitly requested, but some required packages are not available])
-  fi
 fi
 
 SPICE_CHECK_LZ4

Comments

On Tue, 3 Nov 2015, Francois Gouget wrote:

> Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
> ---
>  configure.ac | 4 ----
>  1 file changed, 4 deletions(-)
> 
> This patch does not depend on any other patch in this series.

Anything blocking this?
(there must be a curse...)


> diff --git a/configure.ac b/configure.ac
> index 5790a37..7033cbb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -671,10 +671,6 @@ have_dbus=no
>  if test "x$enable_dbus" != "xno"; then
>    AC_DEFINE([USE_GDBUS], [1], [Define if supporting gdbus])
>    have_dbus=yes
> -
> -  if test "x$enable_dbus" = "xyes" && test "x$have_dbus" = "xno"; then
> -    AC_MSG_ERROR([D-Bus support explicitly requested, but some required packages are not available])
> -  fi
>  fi
>  
>  SPICE_CHECK_LZ4
> -- 
> 2.6.1
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>
> 
> On Tue, 3 Nov 2015, Francois Gouget wrote:
> 
> > Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
> > ---
> >  configure.ac | 4 ----
> >  1 file changed, 4 deletions(-)
> > 
> > This patch does not depend on any other patch in this series.
> 
> Anything blocking this?
> (there must be a curse...)
> 

Was already acked by Christophe, perhaps he forgot to merge it.

Merged!

Thanks,
  Frediano