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
treewide: chown user:group instead of user.group to fix warnings from coreutils 9.1 #171703
Conversation
The .md/.xml inconsistency at https://github.com/NixOS/nixpkgs/runs/6313067719?check_suite_focus=true#step:4:110 seems to be someone else's fault, unless I'm missing something. |
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.
Diff LGTM
doesn't need to target staging |
Reran the manual but still issues, and not seeing them on other PRs. Not sure what's up with that. |
…x warnings from coreutils 9.1 activating the configuration... setting up /etc... chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.messagebus’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ chown: warning: '.' should be ':': ‘root.root’ reloading user units for root...
@Mindavi I changed this to target master, does that help? |
Jep, looks good now. |
mkdir -p /etc/tinc/${network}/invitations | ||
chown tinc.${network} /etc/tinc/${network}/invitations | ||
chown tinc:${network} /etc/tinc/${network}/invitations |
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 not correct. The group is actually called tinc.${network}.
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.
Sorry about that, I will make a PR
The user is actually tinc.${network}, as Mic92 points out in NixOS#171703 (comment) Sorry, I broke this in NixOS#171703 earlier. coreutils 9.1 chown does not complain in this case with a valid dotted user.
Description of changes
This fixes the following warnings when using coreutils 9.1 (#169046) and activating a new NixOS configuration, and also fixes
chown user.group
everywhere else in the tree.Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notescc @Artturin @dasJ @alyssais