Skip to content

Conversation

@stephen-huan
Copy link
Contributor

This reverts a change in 2dc2a54 that I believe is tangential to the issue that the commit addresses. This makes it more consistent with the default value of the 'shell' option and the rest of the file. Personally, I have a different sh on $PATH than /bin/sh (actually, I do not have /bin/sh at all), which is how I noticed the change.

function! s:neovim_unix_run(cmd) abort " {{{1
call system(['sh', '-c', a:cmd])
endfunction
" }}}1
function! s:neovim_unix_capture(cmd) abort " {{{1
return systemlist(['sh', '-c', a:cmd])
endfunction

let s:shell = executable('sh')
\ ? 'sh'
\ : (executable('/usr/bin/sh')
\ ? '/usr/bin/sh' : '/bin/sh')

@lervag
Copy link
Owner

lervag commented Aug 9, 2024

Thanks, I agree that this seems an unintended side effect.

@lervag lervag merged commit 7f6817c into lervag:master Aug 9, 2024
@stephen-huan stephen-huan deleted the binsh branch August 9, 2024 07:47
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