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.

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

Occasional error starting up #99

Closed
davidosomething opened this issue Feb 19, 2019 · 4 comments
Closed

Occasional error starting up #99

davidosomething opened this issue Feb 19, 2019 · 4 comments

Comments

@davidosomething
Copy link

  File "/Users/dotrakoun/.config/pyenv/versions/3.7.0/bin/nvr", line 11, in <module>
    sys.exit(main())
  File "/Users/dotrakoun/.config/pyenv/versions/3.7.0/lib/python3.7/site-packages/nvr/nvr.py", line 561, in main
    nvr.server.close()
AttributeError: 'NoneType' object has no attribute 'close'```
@mhinz
Copy link
Owner

mhinz commented Feb 19, 2019

"Occasional"? :)

  • Does it only happen if nvr itself starts a new nvim process or also when it uses an already existing process?
  • Does it happen with just nvr (without arguments) as well?

mhinz added a commit that referenced this issue Feb 19, 2019
@davidosomething
Copy link
Author

davidosomething commented Feb 20, 2019

i always start with PYTHONWARNINGS=ignore nvr -s (and optionally a filename)
it might be when there's a hanging process (e.g. crashed terminal leaving invalid socket file)

just upgraded to your latest release, will let you know if i see it again

@mhinz
Copy link
Owner

mhinz commented Feb 20, 2019

Maybe PYTHONWARNINGS=default would get us a hint. :)

In the meanwhile I'll check if I can break nvr by giving it stale sockets etc.

@mhinz
Copy link
Owner

mhinz commented Mar 25, 2019

I did another few fixes today. Accessing nvr.server when it's None should not happen anymore.

I also fixed a few warnings that should only happen with PYTHONWARNINGS=default, but some can't be fixed from my side. They need changes in pynvim. But these are unclosed transport warnings and can be ignored.

Please reopen this as soon as you get an issue again.

@mhinz mhinz closed this as completed Mar 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants