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

Reset shell before running a command in go#util#System #967

Merged
merged 2 commits into from
Jul 31, 2016
Merged

Reset shell before running a command in go#util#System #967

merged 2 commits into from
Jul 31, 2016

Conversation

arp242
Copy link
Contributor

@arp242 arp242 commented Jul 27, 2016

This makes sure the commands still work when using a non-Bourne shell such as
fish or tcsh.

:help vim-doc recommends:

    set shell='/bin/sh'

But this is not a great solution. This will mean that :sh will now invoke
/bin/sh, instead of tcsh or fish. It also means that things such as shell
aliases for :! will no longer work.

This makes sure the commands still work when using a non-Bourne shell such as
fish or tcsh.

:help vim-doc recommends:

        set shell='/bin/sh'

But this is not a great solution. This will mean that `:sh` will now invoke
`/bin/sh`, instead of `tcsh` or `fish`. It also means that things such as shell
aliases for `:!` will no longer work.
@fatih fatih merged commit 881a9cc into fatih:master Jul 31, 2016
@fatih
Copy link
Owner

fatih commented Jul 31, 2016

Thanks @Carpetsmoker. This sounds like good improvement.

@arp242
Copy link
Contributor Author

arp242 commented Aug 1, 2016

Thanks @fatih

Maybe it's also a good idea to remove the part in the "TROUBLESHOOTING" section of the help file? This should no longer apply...

@arp242 arp242 deleted the reset-shell branch August 1, 2016 02:03
@fatih
Copy link
Owner

fatih commented Aug 1, 2016

@Carpetsmoker definitely. If you see something that needs to be changed/fixed feel free to open a PR. I'm usually fully loaded with PR's, fixes, my work, my personal time, etc.. So I have limited time. I'm more than happy for contributions 👍

@arp242 arp242 restored the reset-shell branch August 1, 2016 17:44
@arp242 arp242 deleted the reset-shell branch August 1, 2016 17:46
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

Successfully merging this pull request may close these issues.

2 participants