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

No check for existence of sudo command #164

Open
troeger opened this issue Aug 21, 2013 · 1 comment
Open

No check for existence of sudo command #164

troeger opened this issue Aug 21, 2013 · 1 comment

Comments

@troeger
Copy link

troeger commented Aug 21, 2013

Cusine (and Fabric ?) silently assume the 'sudo' command to be available on the machine, which is not given on some minimal Debian setups.

I would propose to add an according check in the initialization step, resulting in a warning. Doing such a check in the sudo() method would indirectly break some package managers, which might be confusing for users.

If needed, I can provide a patch for that.

@sebastien
Copy link
Owner

This is a very good idea. Also, it would be good to check that the shell is SH-compatible, as it's a requirement for Cuisine to work. Looking forward to your pull request!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants