Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@vxfield
Copy link
Member

@vxfield vxfield commented Dec 25, 2020

This PR mitigates issue #737 by using the latest MS Build Locator package (fix).

It picks some changes from the November QDK release fix #755 with:

  • Forcing the Working Directory of the LanguageServer to be the same as the LanguageServer is (in case we might need to add a global.json to select the .NET SDK version)
  • Not quitting the LanguageServer if there was an issue with the MSBuildLocator when .NET Core SDK 3.1 or greater is not found
  • Check if .NET Core SDK 3.1 or greater is not installed and display a message to the user with an option to download it (opens the download page) and asking the user to restart VS/Code.

<PackageReference Include="Newtonsoft.Json" Version="12.0.2" />
<PackageReference Include="StreamJsonRpc" Version="2.2.34" />
<PackageReference Include="System.Reactive" Version="4.2.0" />
<PackageReference Include="NuGet.Frameworks" Version="5.8.0" />
Copy link
Contributor

@bamarsha bamarsha Jan 19, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It worries me a little that it's necessary to include this dependency, since we had issues with depending on a specific version of NuGet.ProjectModel before. @bettinaheim @ricardo-espinoza

vxfield and others added 4 commits January 20, 2021 08:35
Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
@bamarsha
Copy link
Contributor

bamarsha commented Mar 2, 2021

Updating the language server to .NET 5 is blocked until the QDK build pipeline can be updated. I don't know when that will be, so I'll close this for now.

@bamarsha bamarsha closed this Mar 2, 2021
@vxfield vxfield deleted the xfield/build-locator-fix branch January 26, 2022 20:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants