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

Allow override of shell in `local` command #725

Merged
merged 2 commits into from Sep 12, 2012

Conversation

Projects
None yet
2 participants
@mkhattab

mkhattab commented Sep 11, 2012

This patches the local function to allow use of a different shell, i.e.:

local('command', shell="/bin/zsh")

Otherwise, it defaults to whatever shell the underlying platform requires

@bitprophet

This comment has been minimized.

Member

bitprophet commented Sep 12, 2012

Hey, can you add an entry to the top of the changelog for this as well? Don't forget to add your name (see previous entries in the file) for internet kudos :)

@mkhattab

This comment has been minimized.

mkhattab commented Sep 12, 2012

Yes, sorry about that. I should've read the contribution guidelines before submitting. Although it feels strange thanking myself though :)

@bitprophet

This comment has been minimized.

Member

bitprophet commented Sep 12, 2012

I actually just updated those this afternoon, partly because I was going to parrot the same thing to you and went "ok this really needs to be in the docs..." :) so thanks for that too!

bitprophet added a commit that referenced this pull request Sep 12, 2012

Merge pull request #725 from mkhattab/master
Allow override of shell in `local` command

@bitprophet bitprophet merged commit 13a7440 into fabric:master Sep 12, 2012

@bergantine bergantine referenced this pull request Nov 9, 2012

Closed

fabric 1.5 throws errors #16

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