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

[0.4.5] LSP does no longer work correctly on Linux #50

Closed
faliagas opened this issue Feb 16, 2023 · 6 comments
Closed

[0.4.5] LSP does no longer work correctly on Linux #50

faliagas opened this issue Feb 16, 2023 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@faliagas
Copy link

In the Linux version of ecode-0.4.5 the linter does not work. When hovering over a keyword nothing appears. It works OK with version 0.4.4.

@SpartanJ SpartanJ self-assigned this Feb 16, 2023
@SpartanJ SpartanJ added the bug Something isn't working label Feb 16, 2023
@SpartanJ
Copy link
Owner

Thanks for reporting it. Can you give me a more concrete example: language, linter and LSP you're using, also a screenshot would help me, the linting could be from the a linter or the LSP diagnostics. Also, do you see the error but not the description? Is it the linter or the LSP symbol info you're referring to (since the mouse hover behavior could come from both usages).
Thanks

@faliagas
Copy link
Author

Language: C++
LSP: Both clangd and ccls are installed. I don't know which Ecode uses.
Here are some screenshots:

  1. Hovering over a symbol:
  • 0.4.4:
    hover-0 4 4
  • 0.4.5:
    hover-0 4 5
  1. Programming error:
  • 0.4.4:
    programming-error-0 4 4
  • 0.4.5:
    programming-error-0 4 5

@SpartanJ SpartanJ changed the title [0.4.5] Linter does no longer work [0.4.5] LSP does no longer work correctly on Linux Feb 16, 2023
@SpartanJ
Copy link
Owner

Oh, that's the LSP symbol info, it's actually broken in some situations, I sent a fix about an hour ago, let me share a build with the fix, and please tell me if it's fixed for you. Give me some minutes and I'll upload a new build in this ticket.

@SpartanJ
Copy link
Owner

Here's a build with the tentative fix, please let me know if that worked for you: ecode-linux-0.4.5-x86_64.zip. I discovered the same issue this morning while using it, it should be the same issue.

@faliagas
Copy link
Author

Yes, it works!

@SpartanJ
Copy link
Owner

Excellent! Thanks for testing it! I'll keep the issue open until I release a new version, given that this is a breaking issue I'll try to publish a fix this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants