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

Install GitHub.TeamFoundation.15 MEF component in all Visual Studio versions #1831

Closed
wants to merge 2 commits into from

Conversation

jcansdale
Copy link
Collaborator

@jcansdale jcansdale commented Aug 6, 2018

It looks like c9703cd might have stopped the extension from working on some Visual Studio installations.

From the MEF log #1810 (comment), it appears the VSIX installer is attempting to install the GitHub.TeamFoundation.14 assembly on Visual Studio 2017 (and failing), but ignoring the GitHub.TeamFoundation.15 assembly.

What this PR does

  • Allow the GitHub.TeamFoundation.15 assembly to install on all Visual Studio versions

There should no change on Visual Studio 2015 which doesn't support version ranges and will attempt to install all MEF assemblies anyway.

Visual Studio 2017 should always install GitHub.TeamFoundation.15, even if an erroneous attempt has been made to install GitHub.TeamFoundation.14.

We don't know if Visual Studio 2019+ will release new TeamFoundation assemblies. If they don't, this change will allow the extension to carry on working. If they do, we will need to create a new GitHub.TeamFoundation.16 project to target them.

Update: I think this was actually an issue with a reference to Microsoft.VisualStudio.Threading v15.0

Fixes #1810

Install the GitHub.TeamFoundation.15 MEF component in all Visual Studio
versions. This is a workaround for #1810.
Visual Studio 2017 shipped with Microsoft.VisualStudio.Threading 15.0.
Reference a version that is compatible.
@jcansdale
Copy link
Collaborator Author

I think this was actually an issue with a reference to Microsoft.VisualStudio.Threading v15.0.

@jcansdale jcansdale closed this Aug 7, 2018
@jcansdale jcansdale deleted the fixes/1810-MEF-Asset-TargetVersion branch August 7, 2018 15:40
@meaghanlewis meaghanlewis added this to the 2.5.5 milestone Aug 7, 2018
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.

2 participants