rg exits with 1 when ran from within neovim about ripgrep HOT 10 CLOSED

This should be fixed now. rg was being a bit too aggressive with trying to read from stdin. It will now only try if it's given a file or a FIFO.

from ripgrep.

Are you sure your search is taking place in the right directory and actually returns results? I can run :!rg --vimgrep foo in my vim just fine.

It might help to try running :!rg --files to see precisely what it's trying to search.

from ripgrep.

@BurntSushi This is weird..

:!rg --files prints out the files properly

However, :!rg appName returns 1

from ripgrep.

Just to confirm the reported issue, I'm also seeing this behavior with the shell returned 1 from rg where ag works as expected. This is using NeoVim (NVIM v0.1.6-120-g4a6b4bb) on OS X 10.11.6 for what it's worth. When I add the --files flag, it does output the expected list of files, but always exits with 1, as far as my limited testing has shown.

from ripgrep.

Thanks @elasticdog

I'm using NeoVim on Ubuntu 14.04 with oh-my-zsh.

from ripgrep.

Interesting, I can finally reproduce this in neovim (Linux), but vim works fine (Linux, Mac).

Only thing I can think of is that rg thinks its getting stdin piped, so it tried to read it, finds nothing, and reports failure. I'll dig into it.

from ripgrep.

could it be due to this? Quoting:

This is not a bug, it is the new behavior of bang commands: We no longer spawn the program with it's stdout connected to Nvim tty, instead we open a pipe, read output and display to the user. This is the only way the bang commands will be consistent across UIs, so programs designed to be used interactively from the terminal will no longer work from inside nvim.

from ripgrep.

I'm running into something similar trying to add rg to atom-fuzzy-grep.

rg --ignore-case --with-filename --no-heading --column produces output compatible to ag.

Testing with rg --files works fine, but with any other flags, the process exits with code 1. I'm not a Node expert, but I think the library author sets the stdio options for ChildProcess to ignore stdin link.

from ripgrep.

@BurntSushi Awesome! Do you plan to release a new version soon? Thank you! 😄

from ripgrep.

I'd like to get a release out today. I'm just trying to cram as many fixes in as I can. :-)

from ripgrep.