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
Update wording on root install #1143
Conversation
@@ -18,11 +18,11 @@ if [ -z "$USER" ]; then | |||
fi | |||
|
|||
if [ "$(id -u)" -eq 0 ]; then | |||
printf '\e[1;31mwarning: installing Nix as root is not supported by this script!\e[0m\n' | |||
printf '\e[1;31mwarning: although it has some users, installing Nix as root is not supported by this script. This install will continue, but we recommend aborting and installing as a user instead.\e[0m\n' |
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.
uses*
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.
oh - I meant 'users' -- as in "there are some people who use this successfully"
Perhaps add info on how to overcome this? In fact, you have to execute only two commands to be able to install Nix as root https://github.com/elitak/nixos-infect/blob/master/nixos-infect#L158-L159 |
@danbst That's not completely true. For non-root users to be able to use Nix, you must have nix-daemon setup correctly. |
fi | ||
|
||
echo "performing a single-user installation of Nix..." >&2 |
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.
It's still single-user; it's just that single-user happens to be root.
Update the wording in the installer when it warns about installing as root.
Also: move the "installing as single-user" print inside the else-clause so that it's less misleading
This closes #1125