After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 674429 - a logout is required to actually make an account an admin
a logout is required to actually make an account an admin
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: User Accounts
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
3.10
Depends on:
Blocks:
 
 
Reported: 2012-04-19 20:19 UTC by William Jon McCann
Modified: 2015-04-15 14:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
show restart notification when account type changed (1.78 KB, patch)
2014-09-17 12:40 UTC, Ondrej Holy
committed Details | Review

Description William Jon McCann 2012-04-19 20:19:29 UTC
Choosing admin in the list isn't enough you have to logout. Ideally this wouldn't be needed. But if it is we should probably tell the user.
Comment 1 Allan Day 2013-04-19 17:28:41 UTC
This could be done with a notification.

"You have been made an administrator on this system. You need to log out to upgrade your account.

[ Ignore ] [ Log Out ]"
Comment 2 Ondrej Holy 2014-09-17 12:38:57 UTC
(In reply to comment #0)
> Choosing admin in the list isn't enough you have to logout. Ideally this
> wouldn't be needed. But if it is we should probably tell the user.

Why it isn't enough?
Comment 3 Ondrej Holy 2014-09-17 12:40:25 UTC
Created attachment 286368 [details] [review]
show restart notification when account type changed

It can be done same way as for language changes if we really need this...
Comment 4 Bastien Nocera 2015-04-14 11:47:33 UTC
(In reply to Ondrej Holy from comment #2)
> (In reply to comment #0)
> > Choosing admin in the list isn't enough you have to logout. Ideally this
> > wouldn't be needed. But if it is we should probably tell the user.
> 
> Why it isn't enough?

Because PAM only assigns group changes with new logins.
Comment 5 Bastien Nocera 2015-04-14 11:50:06 UTC
Review of attachment 286368 [details] [review]:

There's extra libgd changes.

Rest looks good.
Comment 6 Ondrej Holy 2015-04-15 14:28:49 UTC
Comment on attachment 286368 [details] [review]
show restart notification when account type changed

commit afe0a6226724571f263f9f11f00897bc4458380e