-
Notifications
You must be signed in to change notification settings - Fork 63
Possibly wrong path for fsautocomplete #341
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
Comments
I just found out that the problem arises only if The ambiguity could be because
|
I think I found the problem. (use-package eglot-fsharp
:ensure t
:after fsharp-mode
:config
(setq eglot-fsharp-server-install-dir "~/.dotnet/tools/")) because I wanted to make sure that My suggestion would be to include the hardcoded (defcustom eglot-fsharp-server-install-dir
(concat (locate-user-emacs-file "FsAutoComplete") "/" "netcore")
"Install directory for FsAutoComplete."
:group 'eglot-fsharp
:risky t
:type 'directory) With this, |
Setting `eglot-fsharp-server-install-dir' to nil disables the custom "~/.emacs.d/FsAutoComplete/netcore/" tool path. Fixes fsharp#341
Setting `eglot-fsharp-server-install-dir' to nil disables the custom "~/.emacs.d/FsAutoComplete/netcore/" tool path. Fixes fsharp#341
Fixed in #342 (use-package eglot-fsharp
:ensure t
:after fsharp-mode
:custom (eglot-fsharp-server-install-dir nil)) will use the global user tool path. |
Works like a breeze! Thank you. |
Description
eglot-fsharp--path-to-server
expectsfsautocomplete
to be innetcore
subdir.dotnet tool install
does not install it in that subdir.Repro steps
as recommended in https://www.nuget.org/packages/fsautocomplete.
fsautocomplete
is being installed in~/.dotnet/tools
, not in~/.dotnet/tools/netcore
.Actual behavior
I get the error:
Known workarounds
I think the problem is with the function:
which uses the hardcoded value
netcore/fsautocomplete
.With the help of @protesilaos I modified it to:
which fixed the problem.
I work on Arch Linux with SDK 8. I am not sure if this path is also valid for other configurations. It would probably a good idea to have that value parametric.
Related information
Operating system
Linux version 6.8.9-arch1-1
Emacs version
GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.38, cairo version 1.18.0) of 2023-10-23
.NET Runtime, CoreCLR or Mono Version
8.0.104
The text was updated successfully, but these errors were encountered: