Skip to content

Commit 30bb66a

Browse files
authored
Don't build the composite runtime pack on Windows when we don't ship it (#63724)
R2R composite binaries don't work with binskim. We don't need this on Windows anyway, so don't build it in the first place.
1 parent 063ba34 commit 30bb66a

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

eng/Build.props

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,6 @@
162162
$(RepoRoot)src\Framework\App.Ref\src\Microsoft.AspNetCore.App.Ref.sfxproj;
163163
$(RepoRoot)src\Framework\App.Ref.Internal\src\Microsoft.AspNetCore.App.Ref.Internal.csproj;
164164
$(RepoRoot)src\Framework\App.Runtime\src\aspnetcore-runtime.proj;
165-
$(RepoRoot)src\Framework\App.Runtime\src\aspnetcore-runtime-composite.proj;
166165
$(RepoRoot)src\Framework\App.Runtime\src\aspnetcore-base-runtime.proj;
167166
$(RepoRoot)src\Framework\App.Runtime\bundle\aspnetcore-runtime-bundle.bundleproj;
168167
$(RepoRoot)eng\tools\HelixTestRunner\HelixTestRunner.csproj;
@@ -175,6 +174,9 @@
175174
$(RepoRoot)**\bin\**\*;
176175
$(RepoRoot)**\obj\**\*;"
177176
Condition=" '$(BuildMainlyReferenceProviders)' != 'true' " />
177+
<!-- Only build the composite R2R runtime pack when we will actually ship it -->
178+
<DotNetProjects Include="$(RepoRoot)src\Framework\App.Runtime\src\aspnetcore-runtime-composite.proj"
179+
Condition=" '$(BuildMainlyReferenceProviders)' != 'true' and '$(TargetOsName)' != 'win' " />
178180
<DotNetProjects Include="
179181
$(RepoRoot)src\Assets\**\*.*proj;
180182
$(RepoRoot)src\Caching\**\src\*.csproj;

0 commit comments

Comments
 (0)