Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Conversation

@jcansdale
Copy link
Collaborator

@jcansdale jcansdale commented Aug 7, 2018

What this PR does

  • Fix extension on versions of Visual Studio 2017 earlier than 15.4.4
  • Reference Microsoft.VisualStudio.Shell.15.0 version 15.0.26228
  • Reference Microsoft.VisualStudio.Threading version 15.0.240

This allows the extension to continue working on versions of Visual Studio 2017 earlier than 15.4.4 (#1812 was a report from someone using Visual Studio 15.2.6).

Note, this doesn't fix either of the following (they appear to have been caused by random extension cache corruption):

How to test

Ideally we would be checking that the extension works with the first RTM version of Visual Studio 2017. I don't know if this is still available for download?

Fixes #1812 Error On Project Load (VS 15.2.6)

Fall back to referencing Microsoft.VisualStudio.Threading.15.0. This
should maintain compatibility with versions of Visual Studio 2017 prior
to 15.3.

Microsoft.VisualStudio.Shell.15 has a package dependency constraint and
has been changed from 15.4.27004 to 15.0.26228.
@jcansdale jcansdale changed the title Reference Microsoft.VisualStudio.Threading.15.0 Reference Microsoft.VisualStudio.Threading v15.0 Aug 7, 2018
@jcansdale
Copy link
Collaborator Author

@sharwell I can't for the life of me work out why changing from:

  <package id="Microsoft.VisualStudio.Shell.15.0" version="15.4.27004" targetFramework="net461" />
  <package id="Microsoft.VisualStudio.Threading" version="15.4.4" targetFramework="net461" />

to

  <package id="Microsoft.VisualStudio.Shell.15.0" version="15.0.26228" targetFramework="net461" />
  <package id="Microsoft.VisualStudio.Threading" version="15.0.240" targetFramework="net461" />

Would be necessary for an extension to install correctly on Visual Studo v15.7.6. See:

This fix makes sense for the person running v15.2.6, but it doesn't make sense for any of the others. 😕

Have you come across anything else like this before?

@jcansdale jcansdale changed the title Reference Microsoft.VisualStudio.Threading v15.0 Reference Microsoft.VisualStudio.Shell.15.0 version 15.0.26228 Aug 8, 2018
@grokys
Copy link
Contributor

grokys commented Aug 10, 2018

So does it look like this doesn't actually fix the problem and it's actually the uninstall/reinstall/update process that fixes the problem?

@jcansdale
Copy link
Collaborator Author

@grokys this fixes #1812, which was someone running Visual Studio 15.2.6. The rest were fixed by reinstalling the extension. 😭 I'll update the PR description.

@jcansdale jcansdale changed the title Reference Microsoft.VisualStudio.Shell.15.0 version 15.0.26228 Fix extension on versions of Visual Studio 2017 earlier than 15.4.4 Aug 10, 2018
@jcansdale jcansdale requested a review from meaghanlewis August 10, 2018 15:44
@meaghanlewis
Copy link
Contributor

This LGTM! Tested on VS2017 Version 15.0.26228.48.

I installed the latest release version of GHfVS ( 2.5.4.3349) which was very broken and then installed the build from this PR and all our major functionality is working again.

Thanks @jcansdale 🎉

@jcansdale
Copy link
Collaborator Author

This LGTM! Tested on VS2017 Version 15.0.26228.48.

Excellent! This sounds like a good - worst case - VM to have in reserve. 😄

@grokys grokys merged commit 6c121c1 into master Aug 13, 2018
@grokys grokys deleted the fixes/1812-VisualStudio-Threading-15.0 branch August 13, 2018 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error On Project Load

5 participants