Merge pull request #7774 from dibarbet/patch_release

(Release) Fix .NET acquisition
dotnet · Nov 13, 2024 · 6ca2cfa · 6ca2cfa
2 parents 78595d7 + cc45ccb
commit 6ca2cfa
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/lsptoolshost/dotnetRuntimeExtensionResolver.ts b/src/lsptoolshost/dotnetRuntimeExtensionResolver.ts
@@ -105,15 +105,18 @@ export class DotnetRuntimeExtensionResolver implements IHostExecutableResolver {
             return this.hostInfo;
         }
 
+        // We have to use '8.0' here because the runtme extension doesn't support acquiring patch versions.
+        // The acquisition will always acquire the latest however, so it will be at least 8.0.10.
+        const dotnetAcquireVersion = '8.0';
         let status = await vscode.commands.executeCommand<IDotnetAcquireResult>('dotnet.acquireStatus', {
-            version: DotNetRuntimeVersion,
+            version: dotnetAcquireVersion,
             requestingExtensionId: CSharpExtensionId,
         });
         if (status === undefined) {
             await vscode.commands.executeCommand('dotnet.showAcquisitionLog');
 
             status = await vscode.commands.executeCommand<IDotnetAcquireResult>('dotnet.acquire', {
-                version: DotNetRuntimeVersion,
+                version: dotnetAcquireVersion,
                 requestingExtensionId: CSharpExtensionId,
             });
             if (!status?.dotnetPath) {