Merge lp:~darkxst/ubuntu/trusty/gdm/nokill into lp:ubuntu/trusty/gdm

Proposed by Tim Lunn
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~darkxst/ubuntu/trusty/gdm/nokill
Merge into: lp:ubuntu/trusty/gdm
Diff against target: 28 lines (+7/-3)
2 files modified
debian/changelog (+7/-0)
debian/gdm.postinst (+0/-3)
To merge this branch: bzr merge lp:~darkxst/ubuntu/trusty/gdm/nokill
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+202393@code.launchpad.net

Description of the change

Some recent change in trusty seems to have caused gdm to get killed when upgrading the package which brings down the current session.

This removes the kill command from postinst, to avoid killing the users session.

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks! Uploaded.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-11-26 09:31:17 +0000
3+++ debian/changelog 2014-01-20 23:47:57 +0000
4@@ -1,3 +1,10 @@
5+gdm (3.10.0.1-0ubuntu2) UNRELEASED; urgency=medium
6+
7+ * debian/gdm.postinst:
8+ - Don't kill gdm, since this brings down the current session when upgrading
9+
10+ -- Tim Lunn <tim@feathertop.org> Tue, 21 Jan 2014 10:39:39 +1100
11+
12 gdm (3.10.0.1-0ubuntu1) trusty; urgency=medium
13
14 * New Upstream Release (LP: #1269981)
15
16=== modified file 'debian/gdm.postinst'
17--- debian/gdm.postinst 2013-07-30 20:04:14 +0000
18+++ debian/gdm.postinst 2014-01-20 23:47:57 +0000
19@@ -91,9 +91,6 @@
20 rm -f /etc/pam.d/gdm-welcome
21 fi
22
23-PID=$(status "gdm" 2>/dev/null | awk '/[0-9]$/ { print $NF }')
24-[ -z "$PID" ] || kill -HUP $PID
25-
26 if dpkg-maintscript-helper supports rm_conffile 2>/dev/null; then
27 dpkg-maintscript-helper rm_conffile /etc/X11/Xsession.d/60xdg_path-on-session 3.0.4-0ubuntu4 -- "$@"
28 fi

Subscribers

People subscribed via source and target branches

to all changes: