joaotavora / eglot

A client for Language Server Protocol servers
GNU General Public License v3.0
2.25k stars 200 forks source link

Patch scala metals LSP server executable name #1162

Closed skykanin closed 1 year ago

skykanin commented 1 year ago

It seems that eglot was expecting the scala metals LSP server binary to be named metals-emacs instead of metals. This PR patches this behaviour. At least in nix the metals binary is simply call metals. However if other package managers distribute the binary as metals-emacs as well we could instead do something like:

(scala-mode . ,(eglot-alternatives '("metals" "metals-emacs"))
joaotavora commented 1 year ago

Please send this patch to Eglot upstream at Emacs. lisp/prognodes/eglot.el. the email address is bug-gnu-emacs@gnu.org.