Use arm64 lsp on macoa

Fixes #1780
BetterThanTomorrow · Jun 24, 2022 · b250d6a · b250d6a
1 parent f36a8a2
commit b250d6a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@ Changes to Calva.
 
 - Fix: [Error reporting for load file differs from evaluate code](https://github.com/BetterThanTomorrow/calva/issues/1774)
 - Fix: [Accepting and rejecting Github Copilot suggestions does not work](https://github.com/BetterThanTomorrow/calva/issues/1781)
+- [Use Apple Silicon clojure-lsp builds on M1 and M2 macs when available](https://github.com/BetterThanTomorrow/calva/issues/1780)
 
 ## [2.0.286] - 2022-06-13
 

diff --git a/src/lsp/download.ts b/src/lsp/download.ts
@@ -73,7 +73,11 @@ async function unzipFile(zipFilePath: string, extensionPath: string): Promise<vo
 }
 
 async function downloadClojureLsp(extensionPath: string, version: string): Promise<string> {
-  const artifactName = lspUtil.getArtifactDownloadName();
+  // There were no Apple Silicon builds prior to version 2022.06.22-14.09.50
+  const artifactName =
+    version >= '2022.06.22-14.09.50' || process.platform !== 'darwin'
+      ? lspUtil.getArtifactDownloadName()
+      : lspUtil.getArtifactDownloadName('darwin', 'x64');
   const url =
     version !== 'nightly'
       ? `https://github.com/clojure-lsp/clojure-lsp/releases/download/${version}/${artifactName}`

diff --git a/src/lsp/utilities.ts b/src/lsp/utilities.ts
@@ -7,9 +7,7 @@ const versionFileName = 'clojure-lsp-version';
 const artifacts = {
   darwin: {
     x64: 'clojure-lsp-native-macos-amd64.zip',
-    // Should M1 Macs use emulated native binary or native standalone jar until M1 native available?
-    // For now, let's use the Intel binary
-    arm64: 'clojure-lsp-native-macos-amd64.zip',
+    arm64: 'clojure-lsp-native-macos-aarch64.zip',
   },
   linux: {
     x64: 'clojure-lsp-native-static-linux-amd64.zip',