search

lyuts / vim-rtags

Vim bindings for rtags, llvm/clang based c++ code indexer.
BSD 2-Clause "Simplified" License
282 stars 56 forks source link

Shell syntax error in ExecuteRCAsync #73

Open yshui opened 7 years ago

yshui commented 7 years ago

https://github.com/lyuts/vim-rtags/blob/master/plugin/rtags.vim#L570

First, what you want is "&>" not ">&", which means redirect to file descriptor. Secondly, no, this is not portable. Lastly, you should use the on_stderr, on_stdout callbacks, not temporary files.

lyuts commented 7 years ago

Hi. That's how it is used to be. However, this was causing issues for someone. From what I could tell that was caused by that person's shell (it might have be broken for my shell as well, don't remember exactly), and I did this change https://github.com/lyuts/vim-rtags/commit/74e667e7ebfb533b9e2cff503d4bb2385dd6c450

I agree i should switch to a portable approach.