Launchpad has imported 12 comments from the remote bug at https://bugs.freedesktop.org/show_bug.cgi?id=40020.
If you reply to an imported comment from within Launchpad, your comment will be sent to the remote bug automatically. Read more about Launchpad's inter-bugtracker facilities at https://help.launchpad.net/InterBugTracking. ------------------------------------------------------------------------ On 2011-08-11T16:33:47+00:00 Michael Terry wrote: Hello! I've working on adding support to accountsservice for LightDM, which Ubuntu is switching to starting with Ubuntu 11.10. Currently, accountsservice assumes GDM is in use. (It matters only for saving and loading autologin settings.) This patch will do two things: 1) When saving autologin settings, try to write to both GDM and LightDM config files, if they are available. 2) When loading autologin settings, try to determine which of GDM and LightDM are running and load from its respective config file. Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/12 ------------------------------------------------------------------------ On 2011-08-11T16:43:24+00:00 Rstrode wrote: That's probably okay. Maybe a better approach, though, would be to get away from DM specific code and make accounts service manage its own configuration that the DMs use instead of their own configuration. thoughts? Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/13 ------------------------------------------------------------------------ On 2011-08-11T16:43:30+00:00 Michael Terry wrote: Created attachment 50137 Proposed patch One notable thing about this patch is how it checks whether GDM is running. It currently checks if org.gnome.DisplayManager is owned. Is there a better way? Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/14 ------------------------------------------------------------------------ On 2011-08-11T16:48:16+00:00 Rstrode wrote: (In reply to comment #2) > Created an attachment (id=50137) [details] > Proposed patch > > One notable thing about this patch is how it checks whether GDM is running. > It > currently checks if org.gnome.DisplayManager is owned. Is there a better way? org.gnome.DisplayManager seems reasonable to me. Other than that, are you happy with your patch? Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/15 ------------------------------------------------------------------------ On 2011-08-11T16:52:59+00:00 Michael Terry wrote: As for the patch, I have done some quick tests and it seems to work for me. I'll likely play with it another day or so and make sure I don't hit any issues. So if you want to hold off until I give a more resounding vote of QA confidence, that's reasonable. As for config files, I'm fine with either way (accountsservice tweaking others' files or having its own), but I'm not a main LightDM or GDM developer so can't really speak to how easy a central config file would be to adopt. This is just one config value (autologin user). If that's all that's envisioned, I'm not sure it's worth having a special config file. But if you plan to add more display manager settings to accountsservice, maybe it starts to make sense. Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/16 ------------------------------------------------------------------------ On 2011-08-11T16:55:36+00:00 Rstrode wrote: alright, let's go with this approach for now, but reconsider if we end up moving more options (like include/exclude lists?) or gain interest from other DMs Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/17 ------------------------------------------------------------------------ On 2011-08-15T12:12:23+00:00 Matthias Clasen wrote: Fwiw, I've always considered the current 'gdm support' a hack that should go away when gdm starts talking to accountsservice. It makes a lot more sense for the accountsservice to store this information itself, and for interested DMs to query it. Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/26 ------------------------------------------------------------------------ On 2011-08-18T01:22:42+00:00 Robert Ancell wrote: Hi, As the LightDM maintainer I very much support the idea of having LightDM reading configuration from the accounts service. I am currently gauging interest on if other desktops will adopt AccountsService: http://lists.freedesktop.org/archives/lightdm/2011-August/000051.html Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/27 ------------------------------------------------------------------------ On 2011-09-09T11:06:45+00:00 Martin Pitt wrote: Mike, the patch doesn't apply any more to 0.6.14. Do you mind porting it? Are you happy with how well it works? Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/29 ------------------------------------------------------------------------ On 2011-09-09T15:17:30+00:00 Michael Terry wrote: Created attachment 51002 Patch #2 Here's an updated patch for 0.6.14 (and it fixes a bug with detecting LightDM's autologin status). Works for me now. Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/30 ------------------------------------------------------------------------ On 2011-10-17T19:50:33+00:00 Matthias Clasen wrote: Comment on attachment 51002 Patch #2 Review of attachment 51002: ----------------------------------------------------------------- ::: src/daemon.c @@ +1446,5 @@ > +{ > + if (!save_autologin_gdm (daemon, name, enabled, error)) > + return FALSE; > + if (!save_autologin_lightdm (daemon, name, enabled, error)) > + return FALSE; Hmm, I feel like this should always try both save functions, if we don't want to look for which dm is actually in use. ie it should be something like result = TRUE; result &= save_autologin_gdm(...); result &= save_autologin_lightdm(...); return result; Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/32 ------------------------------------------------------------------------ On 2012-04-30T09:26:23+00:00 Alessio Treglia wrote: Hi all! Any news on this? Reply at: https://bugs.launchpad.net/accountsservice/+bug/798962/comments/33 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/798962 Title: g-c-c "automatic login" settings not working with lightdm To manage notifications about this bug go to: https://bugs.launchpad.net/accountsservice/+bug/798962/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
