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

Add sudo and env PATH to each command #219

Merged
merged 2 commits into from Jul 15, 2013

Conversation

Projects
None yet
2 participants
@mizzy
Owner

mizzy commented Jul 14, 2013

I've fixed the one of the problems heard from @ftnk with this patch.

And this is also the solution for the problem reported by @kui with the comment of #217 .

Please tell me how do you think about this patch, @ftnk and @kui.

@kui

This comment has been minimized.

Show comment
Hide comment
@kui

kui Jul 14, 2013

Contributor

LGTM. With your patch, my specs works just as I intended.

Contributor

kui commented Jul 14, 2013

LGTM. With your patch, my specs works just as I intended.

@mizzy

This comment has been minimized.

Show comment
Hide comment
@mizzy

mizzy Jul 14, 2013

Owner

Thanks a lot, @kui !

Owner

mizzy commented Jul 14, 2013

Thanks a lot, @kui !

mizzy added a commit that referenced this pull request Jul 15, 2013

Merge pull request #219 from serverspec/add-sudo-and-env-PATH-for-eac…
…h-command

Add sudo and env PATH to each command

@mizzy mizzy merged commit fc2a826 into master Jul 15, 2013

1 check passed

default The Travis CI build passed
Details

@mizzy mizzy deleted the add-sudo-and-env-PATH-for-each-command branch Jul 15, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment