From: Stefan Kangas <stefankangas@gmail.com>
To: Brian Leung <leungbk@posteo.net>, 58676@debbugs.gnu.org
Subject: bug#58676: Add nil and lua-language-server to eglot-server-programs
Date: Fri, 21 Oct 2022 00:28:12 -0700 [thread overview]
Message-ID: <CADwFkm=hL4ZGw_T=TFJRz4+YkDSnOQzQB70PKF9Ax5La9Uk0qw@mail.gmail.com> (raw)
In-Reply-To: <875ygdwwhy.fsf@posteo.net>
Brian Leung <leungbk@posteo.net> writes:
> Tags: patch
Thanks!
> From 56f5b196ab2a22f9f77109a2e85367527f0cfc97 Mon Sep 17 00:00:00 2001
> From: Brian Leung <leungbk@posteo.net>
> Date: Wed, 21 Sep 2022 13:38:53 -0700
> Subject: [PATCH 1/2] Add nil to eglot-server-programs
>
> * eglot.el (eglot-server-programs): Add nil.
^^^^^^^^
This should be lisp/progmodes/eglot.el. I would also clarify that "nil"
is the name of a language server here, and not the Lisp symbol.
(BTW, is the language server named "nil"? Weird.)
> ---
> eglot.el | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/eglot.el b/eglot.el
> index 901bf30..49592f6 100644
> --- a/eglot.el
> +++ b/eglot.el
> @@ -218,7 +218,7 @@ language-server/bin/php-language-server.php"))
> . ("digestif"))
> (erlang-mode . ("erlang_ls" "--transport" "stdio"))
> (yaml-mode . ("yaml-language-server" "--stdio"))
> - (nix-mode . ("rnix-lsp"))
> + (nix-mode . ,(eglot-alternatives '("nil" "rnix-lsp")))
> (gdscript-mode . ("localhost" 6008))
> ((fortran-mode f90-mode) . ("fortls"))
> (futhark-mode . ("futhark" "lsp"))
> --
> 2.37.3
Any rationale for why "nil" should be preferred to "rnix-lsp"? It would
be good to add that to the commit message.
> From 5a41365bbace7b1e6533ee2664ee601dbeb36306 Mon Sep 17 00:00:00 2001
> From: Brian Leung <leungbk@posteo.net>
> Date: Sun, 20 Feb 2022 22:21:27 -0800
> Subject: [PATCH 2/2] Add lua-language-server to eglot-server-programs
>
> * eglot.el (eglot-server-programs): Add lua-language-server. It is,
^^^^^^^^ ^^
This should be lisp/progmodes/eglot.el.
Also, our convention is to use two spaces at the end of sentences, also
in commit messages.
> at the time of writing, more actively developed and more popular than
> the lua-lsp server, so prioritize that.
> ---
> eglot.el | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/eglot.el b/eglot.el
> index 49592f6..277e04b 100644
> --- a/eglot.el
> +++ b/eglot.el
> @@ -222,7 +222,8 @@ language-server/bin/php-language-server.php"))
> (gdscript-mode . ("localhost" 6008))
> ((fortran-mode f90-mode) . ("fortls"))
> (futhark-mode . ("futhark" "lsp"))
> - (lua-mode . ("lua-lsp"))
> + (lua-mode . ,(eglot-alternatives
> + '("lua-language-server" "lua-lsp")))
> (zig-mode . ("zls"))
> (css-mode . ,(eglot-alternatives '(("vscode-css-language-server" "--stdio") ("css-languageserver" "--stdio"))))
> (html-mode . ,(eglot-alternatives '(("vscode-html-language-server" "--stdio") ("html-languageserver" "--stdio"))))
> --
> 2.37.3
next prev parent reply other threads:[~2022-10-21 7:28 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-21 5:56 bug#58676: Add nil and lua-language-server to eglot-server-programs Brian Leung
2022-10-21 7:28 ` Stefan Kangas [this message]
2022-10-21 7:55 ` Brian Leung
2022-10-21 8:29 ` Stefan Kangas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CADwFkm=hL4ZGw_T=TFJRz4+YkDSnOQzQB70PKF9Ax5La9Uk0qw@mail.gmail.com' \
--to=stefankangas@gmail.com \
--cc=58676@debbugs.gnu.org \
--cc=leungbk@posteo.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).