Skip to content

Commit

Permalink
Disable ngen for the BuildHost
Browse files Browse the repository at this point in the history
Trying to ngen this produces some errors because we're not shipping
the dependencies that are used for the console-application portion of it
(like parsing command line switches and locating an MSBuild in that
process). But we don't have any need for that, so we can just disable
ngen to silence errors.
  • Loading branch information
jasonmalinowski committed Sep 5, 2023
1 parent 5e24433 commit df96112
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/VisualStudio/Setup/Roslyn.VisualStudio.Setup.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -184,6 +184,10 @@
<ForceIncludeInVSIX>true</ForceIncludeInVSIX>
<AdditionalProperties>TargetFramework=net472</AdditionalProperties>
<PkgDefEntry>BindingRedirect</PkgDefEntry>
<!-- Don't ngen the BuildHost executable. We don't expect it to be used in Visual Studio scenarios, except for third-party extensions that might be using it.
At the moment, it's consumed not as a running executable but rather as a library, and it also has some dependencies for the executable portion, like
command line parsing and MSBuildLocator that we don't expect to be shipping either. -->
<Ngen>false</Ngen>
</ProjectReference>
<ProjectReference Include="..\..\Workspaces\CSharp\Portable\Microsoft.CodeAnalysis.CSharp.Workspaces.csproj">
<Name>CSharpWorkspace</Name>
Expand Down

0 comments on commit df96112

Please sign in to comment.