[Ayatana-commits] [Merge] lp:~robert-ancell/indicator-session/use-xdg-seat-path into lp:indicator-session

2011-09-27 Thread noreply
The proposal to merge lp:~robert-ancell/indicator-session/use-xdg-seat-path 
into lp:indicator-session has been updated.

Status: Needs review = Merged

For more details, see:
https://code.launchpad.net/~robert-ancell/indicator-session/use-xdg-seat-path/+merge/76331
-- 
https://code.launchpad.net/~robert-ancell/indicator-session/use-xdg-seat-path/+merge/76331
Your team ayatana-commits is subscribed to branch lp:indicator-session.

___
Mailing list: https://launchpad.net/~ayatana-commits
Post to : ayatana-commits@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp


Re: [Ayatana-commits] [Merge] lp:~robert-ancell/indicator-session/use-xdg-seat-path into lp:indicator-session

2011-09-21 Thread 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.
-- 
https://code.launchpad.net/~robert-ancell/indicator-session/use-xdg-seat-path/+merge/76331
Your team ayatana-commits is subscribed to branch lp:indicator-session.

___
Mailing list: https://launchpad.net/~ayatana-commits
Post to : ayatana-commits@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp


[Ayatana-commits] [Merge] lp:~robert-ancell/indicator-session/use-xdg-seat-path into lp:indicator-session

2011-09-20 Thread Robert Ancell
Robert Ancell has proposed merging 
lp:~robert-ancell/indicator-session/use-xdg-seat-path into lp:indicator-session.

Requested reviews:
  Indicator Applet Developers (indicator-applet-developers)

For more details, see:
https://code.launchpad.net/~robert-ancell/indicator-session/use-xdg-seat-path/+merge/76331

Don't use depcrecated org.freedesktop.DisplayManager.GetSeatForCookie, use 
XDG_SEAT_PATH instead.  GetSeatForCookie/GetSessionForCookie has been removed 
in LightDM 0.9.8.
-- 
https://code.launchpad.net/~robert-ancell/indicator-session/use-xdg-seat-path/+merge/76331
Your team ayatana-commits is subscribed to branch lp:indicator-session.
=== modified file 'src/users-service-dbus.c'
--- src/users-service-dbus.c	2011-09-08 09:23:55 +
+++ src/users-service-dbus.c	2011-09-21 05:50:31 +
@@ -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.);

___
Mailing list: https://launchpad.net/~ayatana-commits
Post to : ayatana-commits@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ayatana-commits
More help   : https://help.launchpad.net/ListHelp