Merge lp:~robert-ancell/indicator-session/use-xdg-seat-path into lp:indicator-session/0.1

[Robert Ancell]

Don't use depcrecated org.freedesktop.DisplayManager.GetSeatForCookie, use XDG_SEAT_PATH instead. GetSeatForCookie/GetSessionForCookie has been removed in LightDM 0.9.8.

[Robert Ancell (robert-ancell)]

I've put these methods back in for 0.9.8 so I can upgrade lightdm without this patch, but please apply it as I will remove these at some point.

=== modified file 'src/users-service-dbus.c'
--- src/users-service-dbus.c	2011-09-08 09:23:55 +0000
+++ src/users-service-dbus.c	2011-09-21 05:50:31 +0000
@@ -203,40 +203,10 @@
 create_display_manager_proxy (UsersServiceDbus *self)
 {
   UsersServiceDbusPrivate *priv = USERS_SERVICE_DBUS_GET_PRIVATE (self);
-  DBusGProxy *dm_proxy = NULL;
   GError *error = NULL;
-  const gchar *cookie = NULL;
-  gchar *seat = NULL;
+  const gchar *seat = NULL;
   
-  cookie = g_getenv ("XDG_SESSION_COOKIE");
-  if (cookie == NULL || cookie[0] == 0)
-    {
-      g_warning ("Failed to get DisplayManager proxy: XDG_SESSION_COOKIE undefined.");
-      return;
-    }
-
-  dm_proxy = dbus_g_proxy_new_for_name (priv->system_bus,
-                                        "org.freedesktop.DisplayManager",
-                                        "/org/freedesktop/DisplayManager",
-                                        "org.freedesktop.DisplayManager");
-
-  if (!dm_proxy)
-    {
-      g_warning ("Failed to get DisplayManager proxy.");
-      return;
-    }
-
-  /* Now request the proper seat */
-  if (!dbus_g_proxy_call (dm_proxy, "GetSeatForCookie", &error,
-                          G_TYPE_STRING, cookie, G_TYPE_INVALID,
-                          DBUS_TYPE_G_OBJECT_PATH, &seat, G_TYPE_INVALID))
-    {
-      g_warning ("Failed to get DisplayManager seat proxy: %s", error->message);
-      g_object_unref (dm_proxy);
-      g_error_free (error);
-      return;
-    }
-  g_object_unref (dm_proxy);
+  seat = g_getenv ("XDG_SEAT_PATH");
   g_debug ("CREATING DM PROXIES WITH %s", seat);
   priv->display_manager_proxy = dbus_g_proxy_new_for_name (priv->system_bus,
                                                            "org.freedesktop.DisplayManager",
@@ -249,8 +219,6 @@
                                                                  "org.freedesktop.DBus.Properties");
 
 
-  g_free (seat);
-
   if (!priv->display_manager_proxy)
     {
       g_warning ("Failed to get DisplayManager seat proxy.");