Skip to content

Conversation

@rainersigwald
Copy link
Member

This addresses the repro case for #1675, but not its root cause. By
providing a codeBase for the CA assemblies, we ensure that any running
MSBuild can find them when deserializing a log message.

This addresses the repro case for dotnet#1675, but not its root cause. By
providing a `codeBase` for the CA assemblies, we ensure that any running
MSBuild can find them when deserializing a log message.
@rainersigwald
Copy link
Member Author

@mavasani As discussed over email.

I was a bit confused about what assemblies to involve here. I first tried with just CodeAnalysis which seemed to work, but I observed FxCopTask load failures in the Fusion log. So I added that, at which point we crashed again due to failing to load CodeAnalysis.Sdk. So I went with all three. But I don't know if that's right.

@mavasani
Copy link
Contributor

@rainersigwald - that seems correct. FXCopTask assembly can be invoked from MSBuild and has a dependency on CodeAnalysis SDK.

@rainersigwald
Copy link
Member Author

One other thing--the plan for dev15 is that those assemblies are only ever serviced in-place in the VS folder, right? They won't ever be loaded from a NuGet package or something? This would break that.

@mavasani
Copy link
Contributor

Yes, these assemblies are not published via NuGet packages, they ship in the box.

@AndyGerlicher AndyGerlicher merged commit 66a9887 into dotnet:vs15-rtw Feb 14, 2017
@rainersigwald rainersigwald deleted the codeanalysis-codebase branch May 17, 2018 18:43
rainersigwald added a commit to rainersigwald/msbuild that referenced this pull request Feb 5, 2019
These assemblies have moved and versioned in dev16.

Update of the workaround for dotnet#1675 in dotnet#1699.
rainersigwald added a commit that referenced this pull request Feb 7, 2019
These assemblies have moved and versioned in dev16.

Update of the workaround for #1675 in #1699.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants