Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-10-25 Thread Simon McVittie
clone 657751 -1
retitle -1 '/etc/init.d/gdm3 stop' does not terminate Debian-gdm's dbus-daemons 
etc.
severity -1 normal
thanks

On Sun, 02 Sep 2012 at 21:41:21 +0200, Salvatore Bonaccorso wrote:
 I tried to reproduce this via an installation in a VM, login in a user
 and then purging gdm3. The removal went without the problems
 described.

Perhaps you didn't have a complete GNOME system, only gdm3 itself? I can
reproduce this in two ways:

* install the GNOME desktop task in a VM
* switch to tty1 and purge gdm3, gnome-core, gnome and task-gnome-desktop

and

* install the GNOME desktop task in a VM
* log in to X as a user (on tty7)
* user menu - Switch User (moves you to tty8 with a GDM greeter)
* switch to tty1 and purge gdm3, gnome-core, gnome and task-gnome-desktop

Not all of the greeter processes die when /etc/init.d/gdm3 stop is run:

default|archetype% ssh wheezy-desktop ps -U Debian-gdm -u Debian-gdm u~
USER   PID %CPU %MEMVSZ   RSS TTY  STAT START   TIME COMMAND
108   2842  0.0  0.1  30188  1192 ?Ss   21:25   0:00 
/usr/bin/dbus-daemon --fork --print-pid 5 --print-address 7 --session
108   2864  0.0  0.2  53628  2328 ?S21:25   0:00 
/usr/lib/gvfs/gvfsd
108   2870  0.0  0.3 152956  3296 ?Sl   21:25   0:00 
/usr/lib/at-spi2-core/at-spi-bus-launcher
108   2874  0.0  0.1  29920  1508 ?S21:25   0:00 
/usr/bin/dbus-daemon --config-file=/etc/at-spi2/accessibility.conf --nofork 
--print-address 3

 [But anyway, Paul already did an analysis which in any case (even not
 triggered here) should further 'stabilize' the possible purge process
 for gdm3.]

This does seem to fix it; I've uploaded it.

S


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Processed: Re: Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-10-25 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 clone 657751 -1
Bug #657751 [gdm3] gdm3: error on package purge
Bug 657751 cloned as bug 691458
 retitle -1 '/etc/init.d/gdm3 stop' does not terminate Debian-gdm's 
 dbus-daemons etc.
Bug #691458 [gdm3] gdm3: error on package purge
Changed Bug title to ''/etc/init.d/gdm3 stop' does not terminate Debian-gdm's 
dbus-daemons etc.' from 'gdm3: error on package purge'
 severity -1 normal
Bug #691458 [gdm3] '/etc/init.d/gdm3 stop' does not terminate Debian-gdm's 
dbus-daemons etc.
Severity set to 'normal' from 'serious'
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
657751: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=657751
691458: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=691458
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-09-30 Thread intrigeri
Hi,

Salvatore Bonaccorso wrote (02 Sep 2012 19:41:21 GMT) :
 [But anyway, Paul already did an analysis which in any case (even
 not triggered here) should further 'stabilize' the possible purge
 process for gdm3.]

Sergio Villar Senin wrote (03 Sep 2012 13:53:04 GMT) :
 I'm running sid right now, and all I can say is that I am not able
 to reproduce it with the current unstable version
 (gdm3_3.4.1-2_amd64.deb)

I suggest #657751 is cloned, the new bug retitled to improve handling
or userdel output in postrm, its severity set to normal, and the
original #657751 closed. I'm happy to do it if the GNOME maintainers
give their go ahead.

Thoughts?


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-09-03 Thread Sergio Villar Senin
En 02/09/12 21:41, Salvatore Bonaccorso escribiu:
 Hi
 
 Looking at wheezy bugs to be fixed I stumbled over #657751 in
 gdm3. Sergio, are you able to repoduce this with current version in
 wheezy?

I'm running sid right now, and all I can say is that I am not able to
reproduce it with the current unstable version (gdm3_3.4.1-2_amd64.deb)

BR


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-09-02 Thread Salvatore Bonaccorso
Hi

Looking at wheezy bugs to be fixed I stumbled over #657751 in gdm3.
Sergio, are you able to repoduce this with current version in wheezy?

I tried to reproduce this via an installation in a VM, login in a user
and then purging gdm3. The removal went without the problems
described.

[But anyway, Paul already did an analysis which in any case (even not
triggered here) should further 'stabilize' the possible purge process
for gdm3.]

Thoughts?

Regards,
Salvatore


signature.asc
Description: Digital signature


Bug#657751: gdm3: 657751: patch to fix purging when gdm3 is running

2012-03-10 Thread Paul Wise
tags 657751 + patch
thanks

The issue seems to be that userdel doesn't allow removal when a user is
still in use and gdm3 uses the Debian-gdm user to run parts of itself.

The prerm already tries to stop gdm3 when there is no user logged in and
in this case it seems that there is a user logged in so it doesn't stop.

Killing an active user session doesn't seem to be appropriate,
especially since it could be controlling the dpkg/apt process running
the postrm script.

However, that is all moot because the postrm script should not fail
since it ignores any success/failure of deluser.

It appears the stdout and stderr of userdel is leaking into the code
that purges info from the debconf db. Obviously that output is not a
valid command stream for db_purge so it fails.

If we load the debconf confmodule before generating any output, then the
code in the debconf confmodule will handle redirecting stdout/stderr
correctly so that there are no failures, patch below:

--- gdm3-3.0.4/debian/gdm3.postrm
+++ gdm3-3.0.4/debian/gdm3.postrm
@@ -1,6 +1,13 @@
 #!/bin/sh
 set -e
 
+# Need to load this before any output on stdout/stderr
+# otherwise the debconf command-stream will get
+# some invalid input and cause the failure of postrm.
+if [ $1 = purge ]  [ -e /usr/share/debconf/confmodule ]; then
+. /usr/share/debconf/confmodule
+fi
+
 if [ $1 = purge ] ; then
 update-rc.d gdm3 remove /dev/null
 if [ -d /etc/gdm3 ]; then

-- 
bye,
pabs

http://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part