[banshee] build: linking against dbus-glib-1 is now mandatory (bgo#692374)
- From: Andrés Aragoneses <aaragoneses src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [banshee] build: linking against dbus-glib-1 is now mandatory (bgo#692374)
- Date: Tue, 18 Mar 2014 10:22:06 +0000 (UTC)
commit abdfd9c0786d159b7c24cbcff56eabd61f5d6b9a
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 d74ae6e..97202e2 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 123154b..fc2f6c4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -124,7 +124,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 6960e5e..06762fb 100755
--- a/extras/create-release-notes
+++ b/extras/create-release-notes
@@ -88,6 +88,7 @@ $features
<li>gio-sharp >= 2.99.1, gudev-sharp and gkeyfile-sharp</li>
<li>libgpod-sharp >= 0.7.95</li>
<li>Mono.Upnp >= 0.1</li>
+ <li>dbus-glib >= 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 e056768..baa1203 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Gui/GtkBaseClient.cs
@@ -108,11 +108,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 ed2363b..ea2a8f8 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]