-
Notifications
You must be signed in to change notification settings - Fork 33
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
Soloist is incompatible with the homebrew cookbook because it forces you to run as root #27
Comments
Well that's odd. Soloist totally runs as you, and so has the ability to On Saturday, May 18, 2013, Max Edmands wrote:
Doc Ritezel |
@cunnie and I ran into this when we tried to switch Sprout over to It has to do with the way that the homebrew cookbook shells out. At least in the case of Sprout, I think we'll want to keep sudo but find On Saturday, May 18, 2013, Doc Ritezel wrote:
|
@hiremaga, based on your suggestion I locally modified the homebrew cookbook's def brew(*args)
get_response_from_command("brew #{args.join(' ')}")
end to: def brew(*args)
get_response_from_command("su #{node['current_user']} -c 'brew #{args.join(' ')}'")
end And it made everything work like a charm. Seems to go against their "it's anticipated that you'll run the cookbook as your own user" though, so I'm not sure if they'd accept this as a pull request... |
Fast forward to the year 2015. We just ran into this and burned 30 mins or so. |
@camelpunch - advice on how to fix this? I've been unable to get our soloist scripts working because of this issue. |
From the readme for the opscode-cookbooks/homebrew repository:
...and if you use soloist to install any homebrew packages:
I'm new enough to chef that I'm not sure what the best thing to do here is -- but maybe we should make it optional to run soloist as root?
The text was updated successfully, but these errors were encountered: