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

master does not start (file missing) #7722

Closed
sylvain-bougnoux opened this issue Aug 20, 2018 · 10 comments
Closed

master does not start (file missing) #7722

sylvain-bougnoux opened this issue Aug 20, 2018 · 10 comments

Comments

@sylvain-bougnoux
Copy link
Contributor

Today's version crashes on init

Traceback (most recent call last):
  File "D:\Bougnoux\Contrib\spyder\spyder\app\mainwindow.py", line 3211, in main
    mainwindow = run_spyder(app, options, args)
  File "D:\Bougnoux\Contrib\spyder\spyder\app\mainwindow.py", line 3075, in run_spyder
    main.setup()
  File "D:\Bougnoux\Contrib\spyder\spyder\app\mainwindow.py", line 839, in setup
    self.lspmanager.start_lsp_client('python')
  File "D:\Bougnoux\Contrib\spyder\spyder\plugins\lspmanager.py", line 712, in start_lsp_client
    language_client['instance'].start()
  File "D:\Bougnoux\Contrib\spyder\spyder\utils\code_analysis\lsp_client.py", line 133, in start
    creationflags=creation_flags)
  File "c:\program files\python36\lib\subprocess.py", line 709, in __init__
    restore_signals, start_new_session)
  File "c:\program files\python36\lib\subprocess.py", line 997, in _execute_child
    startupinfo)
FileNotFoundError: [WinError 2] file does not exist

Sorry I tried to debug which file but I could not.
Back to last week commit works (eg b34a22a).

BTW the report issue window popped up but disappear instantaneously. I don't know why either. Maybe it is a separate issue?

@jnsebgosselin
Copy link
Member

I think PR #4751 added new dependencies that need to be installed in order for Spyder to run.

Those are coloredlogs, python-language-server[all], pydocstyle and pexpect

After installing those, Spyder is starting correctly.

@ccordoba12 I do not see these new dependencies in requirements.txt or test_requirements.txt, should they be added there?

@jnsebgosselin
Copy link
Member

jnsebgosselin commented Aug 20, 2018

... but then Go to definition and Help is broken for me while it was working perfectly fine before this. So I don't know if its the right thing to do or if there is something else to install.

Let's wait for an answer from @ccordoba12 or @andfoy.

@ccordoba12
Copy link
Member

ccordoba12 commented Aug 20, 2018

We're in the process of adding a new code completion architecture. Things are going to be a bit rough during the next couple of weeks, so we recommend to stay with beta1 until beta2 is released.

By the way, @jnsebgosselin was right about the new packages that need to be installed.

@jnsebgosselin
Copy link
Member

@ccordoba12 Thanks for your answer! Help is working after all, its the go to definition that is not.

I think I was able to pinpoint why it is not working. Should I open a new issue for this or you guys got it all covered already?

@ccordoba12
Copy link
Member

Should I open a new issue for this or you guys got it all covered already?

Yes, please do, but wait until PR #7725 is merged. That's our third huge PR in a row!!

@jnsebgosselin
Copy link
Member

Oh nice!

I've explained what I think the reason of the problem is in Issue #7726. I've created the issue already since I do not think the the split plugin merge will have an impact on this and it was already ready.

@ccordoba12
Copy link
Member

It's not that that will interfere, it's just that all new work needs to be based on the layout defined by that PR.

@ccordoba12
Copy link
Member

Closing because this should be fixed now by #7822.

@sylvain-bougnoux
Copy link
Contributor Author

sylvain-bougnoux commented Sep 11, 2018 via email

@ccordoba12
Copy link
Member

Great!! Thanks for letting us know about it.

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

3 participants