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

Rename neovim to pynvim #26

Closed
wants to merge 2 commits into from
Closed

Rename neovim to pynvim #26

wants to merge 2 commits into from

Conversation

mcchrish
Copy link

I'm getting this error:

[ncm2_core@yarp] Traceback (most recent call last):
[ncm2_core@yarp]   File "/Users/mcchris/.config/nvim/plugged/nvim-yarp/pythonx/yarp.py", line 1, in <module>
[ncm2_core@yarp]     from neovim import attach, setup_logging
[ncm2_core@yarp] ModuleNotFoundError:
[ncm2_core@yarp] No module named 'neovim'
[ncm2_core@yarp] Job is dead. cmd=['/usr/local/bin/python3', '-u', '/Users/mcchris/.config/nvim/plugged/nvim-yarp/pythonx/yarp.py', '/var/folders/fb/nctk069d4rz
83p4b87b4rx_w0000gp/T/nvimAPLsQb/0', 2, 'ncm2_core']

When installing the latest python-client. Solved it by changing the imports. (don't know a lot of python but this seems to solve the issue)

@CandySunPlus
Copy link

merge it, please

@roxma
Copy link
Owner

roxma commented Nov 22, 2018

Why?
According to ncm2/ncm2#102 and https://github.com/neovim/pynvim/tree/master/neovim, import neovim still works

@Shougo
Copy link
Contributor

Shougo commented Nov 22, 2018

@mcchrish You need to remove neovim package and re-install pynvim is needed.
Because the package upgrading doesn't work properly.

@roxma
Copy link
Owner

roxma commented Nov 24, 2018

Considering that import neovim is still available and pynvim was created 7 days ago. I don't want to break the software for those who install python neovim client using the system package manager, which usually results in older version for point release linux distros.

I should probably merge this PR later.

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

Successfully merging this pull request may close these issues.

None yet

4 participants