[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#598867: unblock: xfce4-session/4.6.2-2



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: freeze-exception

Hi,

Please unblock package xfce4-session

xfce4-session (4.6.2-2) unstable; urgency=low

  * debian/patches:
    - 03_gk-start-with-path and 04_gk-dont-check-running-use-start added,
     change the way it starts gnome-keyring so it correctly runs in the
     session and exports environment variables.                 closes: #523322
  * debian/control
    - update standards version to 3.9.1.

These patches are needed for xfce4-session to launch correctly gnome-keyring >=
2.30. They have been in unstable for almost a month, and no new bugs were
reported about them.

Debdiff attached.

unblock xfce4-session/4.6.2-2

Thanks!
Cheers,
Lionel

-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (101, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.35-trunk-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Index: debian/control
===================================================================
--- debian/control	(revision 4072)
+++ debian/control	(working copy)
@@ -8,7 +8,7 @@
  libxfce4util-dev (>= 4.6.0), libxfcegui4-dev (>= 4.6.0), libwnck-dev, 
  x11-xserver-utils, libxfconf-0-dev (>= 4.6.0), libglade2-dev, 
  intltool (>= 0.31)
-Standards-Version: 3.8.4
+Standards-Version: 3.9.1
 Homepage: http://www.xfce.org/
 Vcs-Svn: svn://svn.debian.org/pkg-xfce/desktop/trunk/xfce4-session/
 Vcs-Browser: http://svn.debian.org/wsvn/pkg-xfce/desktop/trunk/xfce4-session/
Index: debian/changelog
===================================================================
--- debian/changelog	(revision 4072)
+++ debian/changelog	(working copy)
@@ -1,3 +1,14 @@
+xfce4-session (4.6.2-2) unstable; urgency=low
+
+  * debian/patches:
+    - 03_gk-start-with-path and 04_gk-dont-check-running-use-start added,
+     change the way it starts gnome-keyring so it correctly runs in the
+     session and exports environment variables.                 closes: #523322
+  * debian/control
+    - update standards version to 3.9.1.
+
+ -- Yves-Alexis Perez <corsac@debian.org>  Sat, 04 Sep 2010 18:42:53 +0200
+
 xfce4-session (4.6.2-1) unstable; urgency=low
 
   [Stefan Ott]
Index: debian/patches/03_gk-start-with-path.patch
===================================================================
--- debian/patches/03_gk-start-with-path.patch	(revision 0)
+++ debian/patches/03_gk-start-with-path.patch	(revision 4293)
@@ -0,0 +1,12 @@
+--- a/xfce4-session/xfsm-compat-gnome.c
++++ b/xfce4-session/xfsm-compat-gnome.c
+@@ -128,7 +128,8 @@
+   error = NULL;
+   argv[0] = GNOME_KEYRING_DAEMON;
+   argv[1] = NULL;
+-  g_spawn_sync (NULL, argv, NULL, G_SPAWN_LEAVE_DESCRIPTORS_OPEN,
++  g_spawn_sync (NULL, argv, NULL,
++                G_SPAWN_SEARCH_PATH | G_SPAWN_LEAVE_DESCRIPTORS_OPEN,
+                 child_setup, NULL,
+                 &sout, NULL, &status, &error);
+ 
Index: debian/patches/04_gk-dont-check-running-use-start.patch
===================================================================
--- debian/patches/04_gk-dont-check-running-use-start.patch	(revision 0)
+++ debian/patches/04_gk-dont-check-running-use-start.patch	(revision 4293)
@@ -0,0 +1,79 @@
+Index: xfce4-session-4.6.2/xfce4-session/xfsm-compat-gnome.c
+===================================================================
+--- xfce4-session-4.6.2.orig/xfce4-session/xfsm-compat-gnome.c	2010-05-31 14:36:01.988123134 +0200
++++ xfce4-session-4.6.2/xfce4-session/xfsm-compat-gnome.c	2010-05-31 14:37:08.787058601 +0200
+@@ -100,34 +100,23 @@
+ static void
+ gnome_keyring_daemon_startup (void)
+ {
+-  const char *old_keyring;
+-
+   GError *error = NULL;
+   gchar  *sout;
+   gchar **lines;
++  gsize   lineno;
+   gint    status;
+   long    pid;
+   gchar  *pid_str;
+   gchar  *end;
+-  char *argv[2];
+-
+-  /* If there is already a working keyring, don't start a new daemon */
+-  old_keyring = g_getenv ("GNOME_KEYRING_SOCKET");
+-  if (old_keyring != NULL && access (old_keyring, R_OK | W_OK) == 0)
+-    {
+-#ifdef HAVE_GNOME_KEYRING
+-      gnome_keyring_daemon_prepare_environment_sync ();
+-#endif
+-      return;
+-    }
+-
++  char *argv[3];
+ 
+   /* Pipe to slave keyring lifetime to */
+   pipe (keyring_lifetime_pipe);
+ 
+   error = NULL;
+   argv[0] = GNOME_KEYRING_DAEMON;
+-  argv[1] = NULL;
++  argv[1] = "--start";
++  argv[2] = NULL;
+   g_spawn_sync (NULL, argv, NULL,
+                 G_SPAWN_SEARCH_PATH | G_SPAWN_LEAVE_DESCRIPTORS_OPEN,
+                 child_setup, NULL,
+@@ -147,20 +136,22 @@
+     {
+       if (WIFEXITED (status) && WEXITSTATUS (status) == 0 && sout != NULL)
+         {
+-          lines = g_strsplit (sout, "\n", 3);
++          lines = g_strsplit (sout, "\n", 0);
+ 
+-          if (lines[0] != NULL && lines[1] != NULL
+-              && g_str_has_prefix (lines[1], "GNOME_KEYRING_PID="))
+-          {
+-            pid_str = lines[1] + strlen ("GNOME_KEYRING_PID=");
+-            pid = strtol (pid_str, &end, 10);
+-
+-            if (end != pid_str)
+-              {
+-                gnome_keyring_daemon_pid = pid;
+-                xfce_putenv (lines[0]);
+-              }
+-          }
++          for (lineno = 0; lines[lineno] != NULL; lineno++)
++            {
++              xfce_putenv (lines[lineno]);
++
++              if (g_str_has_prefix (lines[lineno], "GNOME_KEYRING_PID="))
++                {
++                  pid_str = lines[lineno] + strlen ("GNOME_KEYRING_PID=");
++                  pid = strtol (pid_str, &end, 10);
++                  if (end != pid_str)
++                    {
++                      gnome_keyring_daemon_pid = pid;
++                    }
++                }
++            }
+ 
+           g_strfreev (lines);
+ 
Index: debian/patches/series
===================================================================
--- debian/patches/series	(revision 4072)
+++ debian/patches/series	(working copy)
@@ -1,2 +1,4 @@
 01_correct_shadows.patch
 02_fix-fortune-path.patch
+03_gk-start-with-path.patch
+04_gk-dont-check-running-use-start.patch

Reply to: