GNU bug report logs - #72016
[PATCH] gnu: emacs-lsp-mode: Enable plists.

Previous Next

Full log

Message #8 received at 72016 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: Katherine Cox-Buday <cox.katherine.e+guix <at> gmail.com>, 72016 <at> debbugs.gnu.org,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>,
 Andrew Tropin <andrew <at> trop.in>
Subject: Re: [bug#72016] [PATCH] gnu: emacs-lsp-mode: Enable plists.
Date: Thu, 18 Jul 2024 17:03:34 +0200

Hey Danny!

Danny Milosavljevic <dannym <at> scratchpost.org> skribis:

> * gnu/packages/emacs-xyz.scm (emacs-lsp-mode)[arguments]<#:phases>
> [enable-plists]: New phase.
>
> Change-Id: I5088954b683b23ab51bd5d6adc09dcc9a1c85ec1
> ---
>  gnu/packages/emacs-xyz.scm | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index d5257f16a9..fd04e2432b 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -31169,6 +31169,14 @@ (define-public emacs-lsp-mode
>                (for-each (lambda (f)
>                            (install-file f "."))
>                          (find-files "clients/" "\\.el$"))))
> +          (add-after 'unpack 'enable-plists
> +            (lambda _
> +              (substitute* "lsp-protocol.el"
> +               ;; This is faster, and it's officially recommended,
> +               ;; and it's required by emacs-lsp-booster.
> +               ;; See also:
> +               ;; <https://emacs-lsp.github.io/lsp-mode/page/performance/>.
> +               (("\\(getenv \"LSP_USE_PLISTS\"\\)") "t"))))

I’m know next to nothing about LSP, but that sounds reasonable to me!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.