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

Bug#598867: marked as done (unblock: xfce4-session/4.6.2-2)



Your message dated Fri, 08 Oct 2010 18:29:23 +0200
with message-id <4CAF46E3.1000201@dogguy.org>
and subject line Re: Bug#598867: unblock: xfce4-session/4.6.2-2
has caused the Debian Bug report #598867,
regarding unblock: xfce4-session/4.6.2-2
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
598867: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=598867
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
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

--- End Message ---
--- Begin Message ---
On 02/10/2010 19:30, Lionel Le Folgoc wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: freeze-exception
> 
> Hi,
> 
> Please unblock package xfce4-session
> 

Unblocked.

Regards,

-- 
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/


--- End Message ---

Reply to: