lsp-asm: if executable is found there's no need to install it (#4564)

* lsp-asm: if executable is found there's no need to install it * Update CHANGELOG.org with lsp-asm change
emacs-lsp · Sep 28, 2024 · 667272d · 667272d
1 parent b333977
commit 667272d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.org b/CHANGELOG.org
@@ -17,6 +17,7 @@
   * Replace the per-interface ~(INTERFACE ...)~ pcase forms with a single,
     unified ~(lsp-interface INTERFACE ...)~ form. The per-interface forms are no
     longer generated. *This is a breaking change.* (See #4430.)
+  * If asm-lsp is installed, lsp-asm won't try to download it to cache store
 
 ** 9.0.0
   * Add language server config for QML (Qt Modeling Language) using qmlls.

diff --git a/clients/lsp-asm.el b/clients/lsp-asm.el
@@ -81,7 +81,8 @@ Will update if UPDATE? is t."
  (make-lsp-client
   :new-connection (lsp-stdio-connection
                    #'lsp-asm--server-command
-                   (lambda () (f-exists? lsp-asm-store-path)))
+                   (lambda () (or (executable-find "asm-lsp")
+                                  (f-exists? lsp-asm-store-path))))
   :major-modes lsp-asm-active-modes
   :priority -1
   :server-id 'asm-lsp