Skip to content
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

installing as root does not fail (or: prints an over-severe warning) #1125

Open
jamesbroadhead opened this issue Nov 10, 2016 · 5 comments
Open
Assignees
Labels

Comments

@jamesbroadhead
Copy link
Contributor

jamesbroadhead commented Nov 10, 2016

If installed as root, the following warning is printed:
"warning: installing Nix as root is not supported by this script!"
The install then continues.

As it stands, the warning worries the user without explaining the consequences or likely outcomes.

This warning is either too severe (root installs are supported) or it should be promoted to an error & the install should fail at this check.

Supporting comment: #697 (comment)

Alternately, support root installs: #936

@edolstra
Copy link
Member

It's a warning because some people do use this script to do root installs, but you have to know what you're doing.

@jamesbroadhead
Copy link
Contributor Author

Yep, I understand -- but the wording implies that it won't work.

Perhaps something like:
"warning: although it has some users, installing Nix as root is not supported by this script. Please see for details. This install will continue, but we recommend aborting and installing as a user instead."

@copumpkin
Copy link
Member

Maybe file a PR with the wording clarified?

@stale
Copy link

stale bot commented Mar 5, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the stale label Mar 5, 2021
@stale
Copy link

stale bot commented Apr 19, 2022

I closed this issue due to inactivity. → More info

@stale stale bot closed this as completed Apr 19, 2022
@thufschmitt thufschmitt reopened this Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants