-
Notifications
You must be signed in to change notification settings - Fork 109
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
Sudo prompt should be early #167
Comments
I think the main reason I hesitate to +1 this is I have my |
Configurable? I'm actually doing this already in a wrapper script that we use. |
Another alternative, Give an option to either "install" or print the actually commands / sudoers configuration options (with |
I am not a sudoers guru, but this is what I ended up doing. This could easily be extended to using a
|
@zacheryph @codekitchen @ryansch I've wrote an article about |
Instead of waiting for the nfs stuff to prompt for sudo (when the user's attention may have wandered while waiting) we should prompt the user early with a no-op sudo command (ex:
sudo true
).The text was updated successfully, but these errors were encountered: