Skip to content

Commit 80d6cf7

Browse files
Include all required shims when building in VMR (#61282)
* Include all required shims when building in VMR * Update condition and add a comment
1 parent 4f66f9a commit 80d6cf7

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

eng/tools/GenerateFiles/Directory.Build.targets.in

+4-2
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,8 @@
6464

6565
<ItemGroup>
6666
<!-- Use the same NETCore shared framework as repo built against except when building product code in servicing. -->
67-
<KnownFrameworkReference Update="Microsoft.NETCore.App">
67+
<!-- We cannot use live shims when building tool packs in VMR - only package for current arch is available. -->
68+
<KnownFrameworkReference Update="Microsoft.NETCore.App" Condition=" '$(DotNetBuild)' != 'true' OR '$(PackAsToolShimRuntimeIdentifiers)' == '' ">
6869
<LatestRuntimeFrameworkVersion
6970
Condition=" '%(TargetFramework)' == '${DefaultNetCoreTargetFramework}' ">${MicrosoftNETCoreAppRuntimeVersion}</LatestRuntimeFrameworkVersion>
7071
<TargetingPackVersion
@@ -83,7 +84,8 @@
8384
<RuntimePackRuntimeIdentifiers Condition="'$(DotNetBuild)' == 'true'">%(RuntimePackRuntimeIdentifiers);$(TargetRuntimeIdentifier)</RuntimePackRuntimeIdentifiers>
8485
</KnownFrameworkReference>
8586

86-
<KnownAppHostPack Update="Microsoft.NETCore.App">
87+
<!-- We cannot use live shims when building tool packs in VMR - only package for current arch is available. -->
88+
<KnownAppHostPack Update="Microsoft.NETCore.App" Condition=" '$(DotNetBuild)' != 'true' OR '$(PackAsToolShimRuntimeIdentifiers)' == '' ">
8789
<AppHostPackVersion
8890
Condition=" '%(TargetFramework)' == '${DefaultNetCoreTargetFramework}' ">${MicrosoftNETCoreAppRuntimeVersion}</AppHostPackVersion>
8991
<AppHostRuntimeIdentifiers Condition=" '$(DotNetBuildSourceOnly)' == 'true' ">$(TargetRuntimeIdentifier)</AppHostRuntimeIdentifiers>

src/Tools/Directory.Build.targets

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
<Project>
22
<PropertyGroup Condition=" '$(PackAsTool)' == 'true' ">
33
<!-- Microsoft tool packages are required to target both x64 and x86. -->
4-
<!-- In VMR builds we only use the current RID. -->
5-
<PackAsToolShimRuntimeIdentifiers Condition=" '$(IsShippingPackage)' == 'true' AND '$(DotNetBuild)' != 'true' ">win-x64;win-x86</PackAsToolShimRuntimeIdentifiers>
6-
<PackAsToolShimRuntimeIdentifiers Condition=" '$(IsShippingPackage)' == 'true' AND '$(DotNetBuild)' == 'true' AND '$(TargetOsName)' == 'win' ">$(TargetRid)</PackAsToolShimRuntimeIdentifiers>
4+
<PackAsToolShimRuntimeIdentifiers Condition=" '$(IsShippingPackage)' == 'true' AND ('$(DotNetBuild)' != 'true' OR '$(TargetOsName)' == 'win')">win-x64;win-x86</PackAsToolShimRuntimeIdentifiers>
75
<!-- None of the tool projects are project reference providers. -->
86
<IsProjectReferenceProvider>false</IsProjectReferenceProvider>
97
<!--

0 commit comments

Comments
 (0)