Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intermittent warning with Microsoft.Build.Engine, Version 14.0 #32

Closed
serbentraut opened this issue Aug 28, 2017 · 15 comments
Closed

Intermittent warning with Microsoft.Build.Engine, Version 14.0 #32

serbentraut opened this issue Aug 28, 2017 · 15 comments
Labels
Milestone

Comments

@serbentraut
Copy link

serbentraut commented Aug 28, 2017

Receive an intermittent warning that says: Analyzer assembly \packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.dll' depends on 'Microsoft.Build.Engine, Version=14.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' but it was not found. Analyzers may not run correctly unless the missing assembly is added as an analyzer reference as well.

If I uninstall and re-install the nuget - it goes away temporarily. Any ideas?

@ejohn20
Copy link
Member

ejohn20 commented Aug 28, 2017

I've seen this before as well. Seems very similar to #26. This appears to resolve itself by closing and reopening Visual Studio in some cases.

@serbentraut
Copy link
Author

FYI - this is occurring on VS 2017 ... version 15.3.2 to be precise.

@ejohn20
Copy link
Member

ejohn20 commented Aug 28, 2017

Can you tell me a little bit about the project being scanned? Web / API / Class project? What version of the framework are they targeting?

@serbentraut
Copy link
Author

MVC Web project, .NET 4.6.1

@CFlatWouldbeMinor
Copy link

CFlatWouldbeMinor commented Sep 18, 2017

We have the same issues. Puma scan complains about requiering msbuild V14, but VS 2017 is built on msbuild V15.
Our projects are .net standard, asp.net core
Warning IDE1003 Analyzer assembly 'C:\.nuget\packages\puma.security.rules\1.0.6\analyzers\dotnet\cs\Microsoft.Build.dll' depends on 'Microsoft.Build.Engine, Version=14.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' but it was not found. Analyzers may not run correctly unless the missing assembly is added as an analyzer reference as well. xx.yy(netstandard2.0)

@ejohn20 ejohn20 added the bug label Sep 18, 2017
@ejohn20
Copy link
Member

ejohn20 commented Sep 18, 2017

Thanks, this will need to be investigated. .NET Standard / Core may not function the same way as the traditional framework references.

@oazabir
Copy link

oazabir commented Oct 1, 2017

VS Community 2017, .net core 1.1 shows this error permanently. Tried restarting IDE, uninstall, reinstall - nothing works.

@ejohn20
Copy link
Member

ejohn20 commented Oct 2, 2017

Thanks Omar, this is expected. .NET Core is not currently supported. The analyzer rules are currently targeting .NET 4.5 in order to complete the web.config transformation.

We are planning on supporting netstandard20 (.NET Core 2.0) eventually but will need to recompile the code and do some regression testing.

@felickz
Copy link

felickz commented Oct 4, 2017

Config and markup file rules work in core! @ejohn20 , any timeline for .NET Core support?

@ejohn20
Copy link
Member

ejohn20 commented Oct 5, 2017

Wow, I didn't expect that to work ;) Opening a separate issue for .NET Core testing and support. I'm open to suggestions if anyone wants to do some testing and let us know the right configuration / packages to reference for full .NET Core support.

@felickz
Copy link

felickz commented Oct 7, 2017

#36

@hikeeba
Copy link

hikeeba commented Oct 26, 2017

We are getting this error also. Our project is targeting .NET 4.6.

Even though this error is logged we still get some analysis warnings.

I'm using v1.0.6 when I run this.

Is there something we need to do to remove this warning?

@ejohn20
Copy link
Member

ejohn20 commented Jan 18, 2018

Updates in v2.0 will handle binding redirects and resolving the assemblies. New documentation will cover how to address these issues as well.

@meadisu27
Copy link
Member

Workaround until the next release is to edit projects files to not include the Microsoft.Build.*.dll's as analyzer references.

Remove the following from project file.
<Analyzer Include="..\packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.dll" /> <Analyzer Include="..\packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.Engine.dll" /> <Analyzer Include="..\packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.Framework.dll" /> <Analyzer Include="..\packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.Tasks.Core.dll" /> <Analyzer Include="..\packages\Puma.Security.Rules.1.0.6\analyzers\dotnet\cs\Microsoft.Build.Utilities.Core.dll" />

@ejohn20
Copy link
Member

ejohn20 commented Jul 30, 2018

Closing until after the 2.0 release. We'll see how the latest version of Roslyn handles this.

@ejohn20 ejohn20 closed this as completed Jul 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants