-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Document incompatibility with MSBuildLocator #6
Comments
@hochata thanks for taking time to report that. I'll have a deeper look soon. But just from a quick look at it, looks like it's something bigger than the shim thing. dotnet/roslyn#67383 |
Henrique Sousa ***@***.***> writes:
@hochata thanks for taking time to report that. I'll have a deeper look soon.
But just from a quick look at it, looks like it's something bigger than the shim thing.
dotnet/roslyn#67383
Yes, by the looks of it, their heuristics are not very accurate. I don't
think there is much to do in this plugin, just wait for the library to
improve. Maybe contribute to their discussions.
I just thought it would be nice to at least have it documented it
somewhere :)
|
As reported on #6, some lsp can fail to detect the dotnet installation, not sure its an error of this library or dotnet heuristics itself.
@hochata, I was investigating this further, I found some interesting things:
Now I'll try to see if there is some environment variable missing that could help with this issue. In the meantime I updated the docs as you suggested, let me know if there are more details that would be nice to have on it. |
It seems that the root cause is this microsoft/MSBuildLocator#210 and it seems that csharp-ls uses a version of MSBuildLocator that is >= 1.5.3 Maybe a fix would be to force the library path with the linker, as stated at dotnet/format#1785 Now the odd thing is that, if I manually run |
docs: specify csharp-ls on the warning information As reported on #6, some lsp can fail to detect the dotnet installation, not sure its an error of this library or dotnet heuristics itself.
Last time I tried I got an error with Omnisharp.
I will trry again and report anything I find.
Thanks for taking your time to solve this!
|
I tried using a symlink as stated, but it fails with the same error.
|
Can you provide information on what editor you use and how you configured omnisharp? I wanted to reproduce it on my side. With |
Sorry, the failure is probably another unrelated mistake on my part. I
can't get it to work even with the officially installed SDK.
But this reminds me that Omnisharp doesn't use the .NET SDK, it requires
the Mono one. And from the logs, it also uses Mono MSBuild. Maybe it
doens't get affected by the bug you decribed?
I can try with the Ionide server to see if anything is different.
|
I could get Omnisharp working with ASDF after installing the Mono
version of MSBuild.
I doesn't suffer from the bug you found, but maybe it is because it is
using an unaffected version.
|
Describe the bug
I tried to run
csharp-ls
and failed with a very cryptic error. I turns out the error was caused by the ASDF shim not being the realdotnet
executable.Steps to reproduce
Details here.
Expected behavior
Code-wise there isn't anything to do. ASDF shims and the MSBuildLocator are just incompatible. But maybe this behavior could be documented here to prevent newbies like me from getting lost in the cryptic stack-traces.
The text was updated successfully, but these errors were encountered: