Fix tool detection order for lsp-unzip on windows (#3022)

It caused issues on windows if unzip was found on the PATH
emacs-lsp · Jun 10, 2024 · cfdd642 · cfdd642
1 parent acad044
commit cfdd642
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.org b/CHANGELOG.org
@@ -4,6 +4,7 @@
   * Add SQL support
   * Add support for Meson build system. (~meson-mode~).
   * Add support for go to definition for external files (.dll) in CSharp projects for OmniSharp server.
+  * Fix lsp-unzip on windows when unzip was found on the PATH
 
 ** 9.0.0
   * Add language server config for QML (Qt Modeling Language) using qmlls.

diff --git a/lsp-mode.el b/lsp-mode.el
@@ -8350,7 +8350,10 @@ archive (e.g. when the archive has multiple files)"
   "Unzip script to unzip file.")
 
 (defcustom lsp-unzip-script (lambda ()
-                              (cond ((executable-find "unzip") lsp-ext-unzip-script)
+                              (cond ((and (eq system-type 'windows-nt)
+                                          (executable-find "powershell"))
+                                     lsp-ext-pwsh-script)
+                                    ((executable-find "unzip") lsp-ext-unzip-script)
                                     ((executable-find "powershell") lsp-ext-pwsh-script)
                                     (t nil)))
   "The script to unzip."