Mercurial > pidgin3 / changeset

--- a/configure.ac	Mon May 13 18:44:54 2013 +0200
+++ b/configure.ac	Tue May 14 01:43:55 2013 +0200
@@ -1539,18 +1539,19 @@
 dnl # Check for Secret Service headers
 dnl #######################################################################
 
-AC_ARG_ENABLE(libsecret, [AC_HELP_STRING([--disable-libsecret], [enable Secret Service support])], enable_secret_service=no, enable_secret_service=yes)
+# disabled - see secretservice.c
+#AC_ARG_ENABLE(libsecret, [AC_HELP_STRING([--disable-libsecret], [enable Secret Service support])], enable_secret_service=no, enable_secret_service=yes)
 
-dnl Check for libsecret; if we don't have it, oh well
-if test "x$enable_secret_service" = "xyes" ; then
-	PKG_CHECK_MODULES(SECRETSERVICE, [libsecret-1], [
-		AC_SUBST(SECRETSERVICE_CFLAGS)
-		AC_SUBST(SECRETSERVICE_LIBS)
-		AC_DEFINE(HAVE_SECRETSERVICE, 1, [Define if we have Secret Service.])
-	])
-fi
+#if test "x$enable_secret_service" = "xyes" ; then
+#	PKG_CHECK_MODULES(SECRETSERVICE, [libsecret-1], [
+#		AC_SUBST(SECRETSERVICE_CFLAGS)
+#		AC_SUBST(SECRETSERVICE_LIBS)
+#		AC_DEFINE(HAVE_SECRETSERVICE, 1, [Define if we have Secret Service.])
+#	])
+#fi
 
-AM_CONDITIONAL(ENABLE_SECRETSERVICE, test "x$enable_secret_service" = "xyes")
+#AM_CONDITIONAL(ENABLE_SECRETSERVICE, test "x$enable_secret_service" = "xyes")
+AM_CONDITIONAL(ENABLE_SECRETSERVICE, test "x1" = "x2")
 
 dnl #######################################################################
 dnl # Check for GNOME Keyring headers
@@ -2939,9 +2940,9 @@
 echo Build with GtkSpell support... : $enable_gtkspell
 echo Build with GCR widgets........ : $enable_gcr
 echo
-echo Build with Secret Service..... : $enable_secret_service
 echo Build with GNOME Keyring...... : $enable_gnome_keyring
 echo Build with KWallet............ : $enable_kwallet
+#echo Build with Secret Service..... : $enable_secret_service
 echo
 echo Build with plugin support..... : $enable_plugins
 echo Build with Mono support....... : $enable_mono

--- a/libpurple/plugins/keyrings/secretservice.c	Mon May 13 18:44:54 2013 +0200
+++ b/libpurple/plugins/keyrings/secretservice.c	Tue May 14 01:43:55 2013 +0200
@@ -1,7 +1,6 @@
 /* purple
  * @file secretservice.c Secret Service password storage
  * @ingroup plugins
- * @todo rewrite it with Complete API
  *
  * Purple is the legal property of its developers, whose names are too numerous
  * to list here.  Please refer to the COPYRIGHT file distributed with this
@@ -22,6 +21,19 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02111-1301  USA
  */
 
+#error "This keyring needs some more work (see TODO)"
+
+/* TODO
+ *
+ * This keyring needs some more work, so it will be disabled until its quality
+ * was raised. Some of the pain points:
+ *  - throws a lot of g_warnings
+ *  - it doesn't notify about some backend faults (like access denied), some of
+ *    them are not handled at all
+ *  - it could use libsecret's Complete API
+ *  - code formatting could be better
+ */
+
 #include "internal.h"
 #include "account.h"
 #include "debug.h"