esamattis / slimux

SLIME inspired tmux integration plugin for Vim
http://esa-matti.suuronen.org/blog/2012/04/19/slimux-tmux-plugin-for-vim/
Other
217 stars 52 forks source link

Slimux breaks when `g:slimux_tmux_path` is unset #65

Closed mganjoo closed 7 years ago

mganjoo commented 7 years ago

See https://github.com/epeli/slimux/commit/55a01a2f87dbec6db18f5199cdf5493aba5dca59#commitcomment-19102917 for details. An extraneous newline causes commands to be split up as follows:

/usr/bin/tmux
 -V

instead of

/usr/bin/tmux -V

and the former causes shell errors.

mganjoo commented 7 years ago

I'd be happy to file a PR later if you prefer.

slashfoo commented 7 years ago

@mganjoo thanks for the report, system() call to find vim has a newline, I'm stripping it in the commit above. Can you verify that it solves the issue on your end so we can close?

mganjoo commented 7 years ago

Seems to be working now; thanks!