-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
user-mgmt.xml: extraUsers => users #31941
Conversation
Heads up: |
seems to be an alias introduced 2 years ago: 14321ae |
{ isNormalUser = true; | ||
home = "/home/alice"; | ||
description = "Alice Foobar"; | ||
extraGroups = [ "wheel" "networkmanager" ]; | ||
groups = [ "wheel" "networkmanager" ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one is actually not valid.
@@ -54,7 +54,7 @@ to the user specification.</para> | |||
group named <literal>students</literal> shall exist: | |||
|
|||
<programlisting> | |||
users.extraGroups.students.gid = 1000; | |||
users.groups.students.gid = 1000; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is ok.
Changed extraUsers -> users and one case of extraGroups -> groups in nixos manual chapter 7. According to chatter on IRC these are the proper names for these configuration options nowadays. Modified based on feedback from Jörg Talheim.
I've now modified the commit based on the feedback of @Mic92. |
Thanks! |
Motivation for this change
According to #nixos@freenode these are the proper names for these configuration settings.
Things done
Changed extraUsers => users and extraGroups => groups in nixos manual chapter 7.