Also marking for hardy. This sounds like a good SRU. However, I'm not
happy with the patch yet:

 - It introduces new strings, which needs to be avoided for i18n reasons.
 - Hardcoding "admin" sounds wrong to me. Shouldn't it rather just check 
whether the user/group already exists? The current code does

        } else if (!user && login_exists (login)) {

  So what about changing login_exists() to also check for the group with
the same name? That would solve the issue more generically and avoid
string changes.


** Also affects: gnome-system-tools (Ubuntu Intrepid)
   Importance: Undecided
       Status: New

-- 
Creating user with username 'admin' hoses admin group, sudo config
https://bugs.launchpad.net/bugs/236305
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to gnome-system-tools in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs

Reply via email to