[banshee/stable-2.6] 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/stable-2.6] build: linking against dbus-glib-1 is now mandatory (bgo#692374)
- Date: Tue, 18 Mar 2014 11:18:01 +0000 (UTC)
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 >= 2.22.3, gtk-sharp-beans >= 2.14.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 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]