Different command for nvr

[ysolis]

I had configured neovim in my machine (Ubuntu 16.04 LTS) with neovim-gnome-terminal wrapper, to have a 'gui' for me.

now, I see that neovim-remote call directly 'nvim' (line 78 in nvim/nvim.py file).

can be possible to setup this command with an ENV var for example ? In my case I have 'neovim' command in my path (that call neovim-gnome-terminal wrapper' and I change in this line to 'neovim' command and works.

this can be an ENV variable that if is setup can be used and if is not, we use nvim command

the same thing can be for silent mode, that can be default if an ENV var is setup too.

[mhinz]

I totally see the need for customizing the command used by nvr. Using an environment variable for it also makes sense in this case.

Adding env var support just for silent mode seems a bit overkill to me though. :-)

So, how does support for $NVR_CMD sound?

[ysolis]

sounds good to me :)

P.D. silent mode, can we see this in another issue ?

[mhinz]

$NVR_CMD should work now!

P.D. silent mode, can we see this in another issue ?

Why do you need that as env variable if -s already exists?

[ysolis]

check this, when no neovim is running, this message appears in silent mode (and launch neovim and create /tmp/mvimsocket )

[mhinz]

Oh, ah. Yes.. nvr foo is nvr --remote foo and the --remote flags don't take -s into account yet. All other options do. I'll fix that.

But $NVR_CMD is working as expected?

[mhinz]

The above commit should make -s work as expected, even for the --remote options.

[ysolis]

-s works, but now neovim-remote launch a new nvim console every time, saying that can't connect to /tmp/nvimsocket ... $NVR_CMD set to nvim and neovim-gnome-wrapper, same scenario :(

[mhinz]

Does /tmp/nvimsocket exist and there is a nvim instance running where :echo v:servername is /tmp/nvimsocket?

[ysolis]

my mistake, I quit neovim and after this /tmp/nvimscket remains, so I delete the socket, restart with nvr -s and it works!

[mhinz]

Oh, usually nvim should remove the socket on its own, except when it got killed.

Anyway, I guess this can be closed now?

[ysolis]

yes :) close this one ... thanks!

P.D. one last thing, add the $NVR_CMD info in the README.rst file.

[mhinz]

add the $NVR_CMD info in the README.rst file

Done.

Thanks for bringing up all these issues!