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

LSP server won't start for current official Godot Server builds #517

Open
turbohz opened this issue Oct 20, 2023 · 4 comments
Open

LSP server won't start for current official Godot Server builds #517

turbohz opened this issue Oct 20, 2023 · 4 comments
Labels

Comments

@turbohz
Copy link

turbohz commented Oct 20, 2023

Godot version

3.6.beta3.official.21ab700f2

VS Code version

1.83.1

Godot Tools VS Code extension version

1.3.1 (master)

System information

Linux

Issue description

This issue is related to: #516

I found an issue with current Godot Server builds, which causes --version (also --help) command to return an error status code (255). This is caught by the plugin check for a valid Godot executable, which then refuses to start the LSP.

This is clearly an issue with Godot, and I will report it as such (if it's not a known issue).

Steps to reproduce

Download a current Godot Server build (platform is listed as "Linux Server").

Run it to obtain the version: Godot_v3.6-beta3_linux_headless.64 --version

You'll see the version printed. However, the status code will be error (255)

@turbohz turbohz added the bug label Oct 20, 2023
@turbohz
Copy link
Author

turbohz commented Oct 20, 2023

Here's where the issue was introduced:

godotengine/godot#62618

@DaelonSuzuka
Copy link
Collaborator

Thanks for tracking down the cause of this. I'll see if I can rearrange the version check to get around this false-negative.

@turbohz
Copy link
Author

turbohz commented Oct 20, 2023

Thanks for tracking down the cause of this. I'll see if I can rearrange the version check to get around this false-negative.

I don't know if it's worth fixing. I don't expect many people to try to use the godot server build to run the LSP server, instead of the editor.

In any case, I submitted a fix and it's been approved (godotengine/godot#83664).

@turbohz
Copy link
Author

turbohz commented Nov 14, 2023

godotengine/godot#83664 has been merged, so it seems this issue will be fixed in the next Godot 3.x release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants