[banshee/stable-2.6] build: linking against dbus-glib-1 is now mandatory (bgo#692374)



commit 5e2c69b659c1b5642d149ce8a61ca2d6c3fd5605
Author: Andrés G. Aragoneses <knocte gmail com>
Date:   Tue Mar 18 01:10:06 2014 +0100

    build: linking against dbus-glib-1 is now mandatory (bgo#692374)
    
    The work-around for this bug was not applied successfully by some
    packagers because we didn't make the dependency mandatory. Now that
    gsettings branch is almost ready to be merged to master (which
    means this workaround would be reverted), it's good if we make sure
    that at least the last GConf-enabled versions don't suffer this bug.

 README                                             |    1 +
 build/m4/banshee/gconf.m4                          |    6 ++----
 configure.ac                                       |    1 -
 extras/create-release-notes                        |    1 +
 .../Banshee.Gui/GtkBaseClient.cs                   |    2 --
 src/Core/Banshee.ThickClient/Makefile.am           |    4 ----
 6 files changed, 4 insertions(+), 11 deletions(-)
---
diff --git a/README b/README
index bd70bf4..fa134b1 100644
--- a/README
+++ b/README
@@ -44,6 +44,7 @@ acceptable; higher versions are fine.
   * gkeyfile-sharp      0.1
   * libgpod-sharp       0.7.95
   * mono-upnp           0.1
+  * dbus-glib           0.80
 
 - Required at run-time:
   * udev
diff --git a/build/m4/banshee/gconf.m4 b/build/m4/banshee/gconf.m4
index fbf1ce6..d7388e1 100644
--- a/build/m4/banshee/gconf.m4
+++ b/build/m4/banshee/gconf.m4
@@ -12,9 +12,7 @@ AC_DEFUN([BANSHEE_CHECK_GCONF],
 
        # dbus-glib is needed for the workaround for bgo#692374
        PKG_CHECK_MODULES(DBUS_GLIB, dbus-glib-1 >= 0.80, have_dbus_glib="yes", have_dbus_glib="no")
-       if test "x$have_dbus_glib" = "xyes"; then
-               AM_CONDITIONAL(HAVE_DBUS_GLIB, true)
-       else
-               AM_CONDITIONAL(HAVE_DBUS_GLIB, false)
+       if test "x$have_dbus_glib" = "xno"; then
+               AC_MSG_ERROR([Please install dbus-glib-1 development package or use --disable-gnome.])
        fi
 ])
diff --git a/configure.ac b/configure.ac
index e9c34b1..d6cabc9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -145,7 +145,6 @@ if test "x$enable_gnome" = "xyes"; then
        AM_CONDITIONAL(ENABLE_GNOME, true)
 else
        AM_CONDITIONAL(GCONF_SCHEMAS_INSTALL, false)
-       AM_CONDITIONAL(HAVE_DBUS_GLIB, false)
        AM_CONDITIONAL(ENABLE_GNOME, false)
 fi
 
diff --git a/extras/create-release-notes b/extras/create-release-notes
index bfe0816..0af02de 100755
--- a/extras/create-release-notes
+++ b/extras/create-release-notes
@@ -89,6 +89,7 @@ $features
        <li>gio-sharp &gt;= 2.22.3, gtk-sharp-beans &gt;= 2.14.1, gudev-sharp and gkeyfile-sharp</li>
        <li>libgpod-sharp &gt;= 0.7.95</li>
        <li>Mono.Upnp &gt;= 0.1</li>
+       <li>dbus-glib &gt;= 0.80</li>
 </ul>
 </li>
        <li>Run-time requirements for default feature stack:
diff --git a/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs 
b/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs
index c3891c1..599bf1b 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs
@@ -112,11 +112,9 @@ namespace Banshee.Gui
                 GLib.Thread.Init ();
             }
 
-#if HAVE_DBUS_GLIB
             // Using GConf from multiple threads causes crashes if multithreading is not initialized 
explictly in dbus
             // This is a workaround for bgo#692374
             dbus_g_thread_init ();
-#endif
 
             Gtk.Application.Init ();
 
diff --git a/src/Core/Banshee.ThickClient/Makefile.am b/src/Core/Banshee.ThickClient/Makefile.am
index ef2af92..3e7a0ff 100644
--- a/src/Core/Banshee.ThickClient/Makefile.am
+++ b/src/Core/Banshee.ThickClient/Makefile.am
@@ -3,10 +3,6 @@ TARGET = library
 ASSEMBLY_BUILD_FLAGS = -unsafe
 LINK = $(REF_BANSHEE_THICKCLIENT)
 
-if HAVE_DBUS_GLIB
-BUILD_DEFINES = "-define:HAVE_DBUS_GLIB"
-endif
-
 SOURCES =  \
        Banshee.Addins.Gui/AddinView.cs \
        Banshee.CairoGlyphs/BansheeLineLogo.cs \


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]