search

Trevoke / alchemist.el

Elixir Tooling Integration Into Emacs
http://www.alchemist-elixir.org
5 stars 2 forks source link

set extension for elixir-ls based on OS #12

Closed erikbye closed 6 years ago

erikbye commented 6 years ago

We could just set the extension to .sh by default, and only explicitly set it to .bat if you're on Windows. But, if you ever want to raise an error and let the user know his OS is not supported, I guess this would be a good place to do it, anyway.

Not even sure extension needs to be specified, at least on Windows, this works fine:

(defun alchemist--lsp-server-path-for-current-project ()
  `(,(concat alchemist-server-root-path
           "erl"
           (alchemist--server-erlang-version (alchemist-project-root-or-default-dir))
           "/"
           "language_server")))

Haven't tested on macOS/Linux, yet.

Trevoke commented 6 years ago

Oh wow, I could have sworn I'd merged this in, well, almost a month ago! Sorry about the massive delay. Thank you for the contribution!