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

the vscode extension should be more lax in finding the v-analyzer executable #10

Open
spytheman opened this issue Jan 11, 2024 · 1 comment
Labels
bug Something isn't working VS Code Extension
Milestone

Comments

@spytheman
Copy link
Member

Describe the bug

The readme suggests using this:

{
  "v-analyzer.serverPath": "/path/to/v-analyzer/bin/v-analyzer"
}

and that is fine on POSIX systems, since the executable files there, do not depend on a specific extension to run.

On Windows though, using that full path, also needs the .exe extension, and that is not mentioned in the README at all, and is not very intuitive to deduce.

Expected Behavior

The vscode extension, should be more flexible, and allow for both /path/v-analyzer and /path/v-analyzer.exe, and also v-analyzer and v-analyzer.exe, if they are in PATH.

Current Behavior

Using the full path, but without .exe on windows systems fails with v-analyzer is not installed, which is too sparse, and can not be used to diagnose the actual problem.

Reproduction Steps

Put just an absolute path to the v-analyzer executable in your settings file, for "v-analyzer.serverPath", but without .exe in it.

Possible Solution

No response

Additional Information/Context

No response

Environment details (v doctor output)

Native windows, the rest does not matter, since the problem affects the extension.

Editor name

vscodium

v-analyzer Version

v-analyzer version 0.0.4-beta.1.1814be2

VS Code Extension Version

vscode-v-analyzer-0.0.4.vsix

@spytheman spytheman added the bug Something isn't working label Jan 11, 2024
@spytheman
Copy link
Member Author

A good addition will be also this:

katekyy — Днес в 14:13
and then change the path in the config so it doesn't need to search all the variants all over again

@ttytm ttytm added this to the 0.x milestone Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working VS Code Extension
Projects
None yet
Development

No branches or pull requests

2 participants