Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

nvr echos the filename when opening a file #153

Open
astier opened this issue Jul 14, 2020 · 3 comments · May be fixed by #182
Open

nvr echos the filename when opening a file #153

astier opened this issue Jul 14, 2020 · 3 comments · May be fixed by #182

Comments

@astier
Copy link

astier commented Jul 14, 2020

Problem

When I open a file with nvr the filename is echoed in nvims-commandline as can be seen in the screenshot. Nvim doesn't do that. Is it possible to suppress it somehow?

Steps

touch foo
nvr foo

Screenshot

nvr_echoes_filename
@astier
Copy link
Author

astier commented Jul 14, 2020

I think nvr should behave by default like nvim.

@astier
Copy link
Author

astier commented Jul 14, 2020

The reason for this behavior is that the nvim setting shortmess is hardcoded and can be found here:

neovim-remote/nvr/nvr.py

Line 99 in 1ec7c6c

shortmess = self.server.options['shortmess']

If I remove all the shortmess-related code nvr behaves just as nvim in this regard.
Is there any particular reason why shortmess is hardcoded? If somebody wants to set it s/he can configure it in their vimrc or pass it as an argument to nvr.

@astier astier mentioned this issue Jul 14, 2020
Closed
@Shougo
Copy link

Shougo commented Nov 19, 2021

@mhinz Why shortmess is ignored?

JRasmusBm added a commit to JRasmusBm/neovim-remote that referenced this issue Sep 3, 2022
@JRasmusBm
fix(shortmess): remove hardcoded workaround
f3840b4 
**Why** is the change needed?

Based on my understanding of
mhinz#92 (comment),
this workaround is no longer needed. Being able to set `shortmess` is
useful for example when running in tmux.

Closes mhinz#153
@JRasmusBm JRasmusBm linked a pull request Sep 3, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(shortmess): remove hardcoded workaround JRasmusBm/neovim-remote
Do not hardcode shortmess #153
2 participants
@Shougo @astier