Merge lp:~darkxst/user-setup/noTimedLogin into lp:~ubuntu-core-dev/user-setup/ubuntu

Proposed by Tim Lunn
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 285
Merged at revision: 285
Proposed branch: lp:~darkxst/user-setup/noTimedLogin
Merge into: lp:~ubuntu-core-dev/user-setup/ubuntu
Diff against target: 36 lines (+9/-5)
2 files modified
debian/changelog (+7/-0)
user-setup-apply (+2/-5)
To merge this branch: bzr merge lp:~darkxst/user-setup/noTimedLogin
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Review via email: mp+256239@code.launchpad.net

Description of the change

There is no reason to set TimedLogin under gdm, for automatic logins, maybe these were broken at some point in GNOME2 days and required both settings but thats not the case for a long time. Further there is no UI in GNOME 3 to disable TimedLogin, so we really don't want it set. Right now anyone who selects Auto Login in ubiquity will need to manually edit config files to disable the TimedLogin

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks good to me, approved.

review: Approve
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

What about Ubuntu Mate? Does that still use GNOME2 stuff that needs TimedLogin?

Revision history for this message
Tim Lunn (darkxst) wrote :

Ubuntu Mate are using LightDM, not gdm, so I don't see anyway this could affect them.

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 2014-09-16 12:56:09 +0000
3+++ debian/changelog 2015-04-14 22:04:23 +0000
4@@ -1,3 +1,10 @@
5+user-setup (1.48ubuntu4) vivid; urgency=medium
6+
7+ * Don't set TimedLogin, this gets set when Auto login is selected
8+ in ubiquity but there is no UI to unset it (LP: #1412791)
9+
10+ -- Tim Lunn <tim@feathertop.org> Tue, 14 Apr 2015 17:44:33 +1000
11+
12 user-setup (1.48ubuntu3) utopic; urgency=medium
13
14 [ KDEUSER56 / Rohan Garg ]
15
16=== modified file 'user-setup-apply'
17--- user-setup-apply 2014-09-16 12:55:05 +0000
18+++ user-setup-apply 2015-04-14 22:04:23 +0000
19@@ -266,15 +266,12 @@
20 cp "$GDMCustomFile" "${GDMCustomFile}$BACKUP"
21 fi
22 AutologinParameters="AutomaticLoginEnable=true\n\
23-AutomaticLogin=$USER\n\
24-TimedLoginEnable=true\n\
25-TimedLogin=$USER\n\
26-TimedLoginDelay=10"
27+AutomaticLogin=$USER\n\"
28
29 # Prevent from updating if parameters already present (persistent usb key)
30 if ! `grep -qs "AutomaticLogin=$USER" $GDMCustomFile` ; then
31 if [ -e "$GDMCustomFile" ]; then
32- sed -i '/\(Automatic\|Timed\)Login/d' $GDMCustomFile
33+ sed -i '/\(Automatic\)Login/d' $GDMCustomFile
34 fi
35 if ! `grep -qs '\[daemon\]' $GDMCustomFile` ; then
36 echo '[daemon]' >> $GDMCustomFile

Subscribers

People subscribed via source and target branches

to all changes: