From e32d7b405472c5bd2ff3ab8e1ce2835da8c4da54 Mon Sep 17 00:00:00 2001
From: Doug Bunting <6431421+dougbu@users.noreply.github.com>
Date: Mon, 20 Apr 2020 09:31:55 -0700
Subject: [PATCH 1/4] Update SDK to preview.5.20251.2 - includes support for
 the `net5.0` TFM - includes fix for `$(PublishDepsFilePath)` - includes fix
 for incorrect NU5118 warnings when `'$(PackAsTool)' == 'true'`

---
 global.json | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/global.json b/global.json
index 7dc4e1b46f1a..dee19df73dff 100644
--- a/global.json
+++ b/global.json
@@ -1,9 +1,9 @@
 {
   "sdk": {
-    "version": "5.0.100-preview.2.20120.3"
+    "version": "5.0.100-preview.5.20251.2"
   },
   "tools": {
-    "dotnet": "5.0.100-preview.2.20120.3",
+    "dotnet": "5.0.100-preview.5.20251.2",
     "runtimes": {
       "dotnet/x64": [
         "$(MicrosoftNETCoreAppInternalPackageVersion)"

From d3caf937040ad3845af5fd33628f5f5b66da6c21 Mon Sep 17 00:00:00 2001
From: Doug Bunting <6431421+dougbu@users.noreply.github.com>
Date: Mon, 20 Apr 2020 15:42:51 -0700
Subject: [PATCH 2/4] !!temporary!! Suppress NU5129 warnings - build/ and
 buildTransitive/ naming conventions do not treat TFMs as equivalent

---
 ...Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.csproj | 5 ++++-
 .../Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.csproj | 6 ++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/Mvc/Mvc.Razor.RuntimeCompilation/src/Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.csproj b/src/Mvc/Mvc.Razor.RuntimeCompilation/src/Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.csproj
index 5e95e71c89f5..62a1e9eece79 100644
--- a/src/Mvc/Mvc.Razor.RuntimeCompilation/src/Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.csproj
+++ b/src/Mvc/Mvc.Razor.RuntimeCompilation/src/Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation.csproj
@@ -1,9 +1,12 @@
 <Project Sdk="Microsoft.NET.Sdk">
-
   <PropertyGroup>
     <Description>Runtime compilation support for Razor views and Razor Pages in ASP.NET Core MVC.</Description>
     <TargetFramework>$(DefaultNetCoreTargetFramework)</TargetFramework>
     <NoWarn>$(NoWarn);CS1591</NoWarn>
+
+    <!-- !!! Temporary workaround while we switch TFMs. -->
+    <NoWarn>$(NoWarn);NU5129</NoWarn>
+
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>aspnetcore;aspnetcoremvc;razor</PackageTags>
     <IsPackable>true</IsPackable>
diff --git a/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.csproj b/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.csproj
index 412ba505cf83..1df08431756a 100644
--- a/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.csproj
+++ b/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.csproj
@@ -1,9 +1,12 @@
 <Project Sdk="Microsoft.NET.Sdk">
-
   <PropertyGroup>
     <Description>Support for writing functional tests for MVC applications.</Description>
     <TargetFramework>$(DefaultNetCoreTargetFramework)</TargetFramework>
     <NoWarn>$(NoWarn);CS1591</NoWarn>
+
+    <!-- !!! Temporary workaround while we switch TFMs. -->
+    <NoWarn>$(NoWarn);NU5129</NoWarn>
+
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>aspnetcore;aspnetcoremvc;aspnetcoremvctesting</PackageTags>
     <IsPackable>true</IsPackable>
@@ -21,5 +24,4 @@
   <ItemGroup>
     <Content Include="Microsoft.AspNetCore.Mvc.Testing.targets" Pack="true" PackagePath="build/$(TargetFramework)/" />
   </ItemGroup>
-
 </Project>

From cc1c9928a5e9c36de58b183138bb91211a2f31c2 Mon Sep 17 00:00:00 2001
From: Doug Bunting <6431421+dougbu@users.noreply.github.com>
Date: Fri, 1 May 2020 15:40:29 -0700
Subject: [PATCH 3/4] Run `InjectRequestHandlerOnPublish` later - late enough
 for .deps file to exist i.e. after the `CopyFilesToPublishDirectory` target -
 `AfterTargets` is a bit flaky; use `BeforeTargets` with empty targets instead

---
 src/Servers/IIS/build/testsite.props | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/Servers/IIS/build/testsite.props b/src/Servers/IIS/build/testsite.props
index 4ff733d3871d..aacd1d8b27e5 100644
--- a/src/Servers/IIS/build/testsite.props
+++ b/src/Servers/IIS/build/testsite.props
@@ -59,17 +59,27 @@
     </PropertyGroup>
   </Target>
 
+  <!--
+    GenerateBuildDependencyFile creates the .deps file used here. BuildDepsJson and CoreBuild are empty targets marking
+    the end of processes using the GenerateBuildDependencyFile target.
+  -->
   <Target Name="InjectRequestHandler"
           Condition=" '$(InProcessTestSite)' == 'true' AND '$(BuildIisNativeProjects)' == 'true' AND '$(TargetFrameworkIdentifier)' != '.NETFramework'"
-          AfterTargets="GenerateBuildDependencyFile"
-          DependsOnTargets="PrepareInjectionApp">
+          BeforeTargets="BuildDepsJson;CoreBuild"
+          DependsOnTargets="GenerateBuildDependencyFile;PrepareInjectionApp">
     <Exec Command="$(InjectDepsApp) &quot;$(ProjectDepsFilePath)&quot; $(RuntimeIdentifier) " />
   </Target>
 
+  <!--
+    GeneratePublishDependencyFile creates an intermediate .deps file copied soon after into the publish folder.
+    CopyFilesToPublishDirectory is an empty target marking the end of all copies into the publish folder, including the
+    .deps file and native assets. Publish is an empty target marking the end of the entire process.
+    (GenerateSingleFileBundle also uses the intermediate .desp file but only in single-file publish cases ignored here.)
+  -->
   <Target Name="InjectRequestHandlerOnPublish"
           Condition=" '$(InProcessTestSite)' == 'true' AND '$(BuildIisNativeProjects)' == 'true' AND '$(TargetFrameworkIdentifier)' != '.NETFramework'"
-          AfterTargets="GeneratePublishDependencyFile"
-          DependsOnTargets="PrepareInjectionApp">
+          BeforeTargets="Publish"
+          DependsOnTargets="GeneratePublishDependencyFile;CopyFilesToPublishDirectory;PrepareInjectionApp">
     <Exec Command="$(InjectDepsApp) &quot;$(PublishDepsFilePath)&quot; $(RuntimeIdentifier) " />
   </Target>
 

From 59fa0dc0a6a4580453e9a1ee7ed20adf06c502f2 Mon Sep 17 00:00:00 2001
From: Doug Bunting <6431421+dougbu@users.noreply.github.com>
Date: Mon, 4 May 2020 13:13:22 -0700
Subject: [PATCH 4/4] Always add `@(AssemblyAttribute)` items before they are
 processed - use `BeforeTargets="GetAssemblyAttributes"` for consistently
 throughout the repo

---
 .../src/Microsoft.AspNetCore.Mvc.Testing.targets            | 6 +++---
 .../BlazorTemplates.Tests/BlazorTemplates.Tests.csproj      | 2 +-
 src/ProjectTemplates/test/ProjectTemplates.Tests.csproj     | 2 +-
 src/Shared/E2ETesting/E2ETesting.targets                    | 6 +++---
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.targets b/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.targets
index 452fc9909d8f..8e9f7093e19c 100644
--- a/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.targets
+++ b/src/Mvc/Mvc.Testing/src/Microsoft.AspNetCore.Mvc.Testing.targets
@@ -23,7 +23,7 @@
       </ItemGroup>
     </Target>
 
-  <Target Name="_AddContentRootForProjectReferences" BeforeTargets="BeforeCompile" DependsOnTargets="_ResolveMvcTestProjectReferences">
+  <Target Name="_AddContentRootForProjectReferences" BeforeTargets="GetAssemblyAttributes" DependsOnTargets="_ResolveMvcTestProjectReferences">
     <ItemGroup>
       <WebApplicationFactoryContentRootAttribute
         Condition="'%(_ContentRootProjectReferences.Identity)' != ''"
@@ -35,7 +35,7 @@
     </ItemGroup>
 
     <ItemGroup>
-      <AssemblyAttribute 
+      <AssemblyAttribute
         Condition=" '%(WebApplicationFactoryContentRootAttribute.Identity)' != '' "
         Include="Microsoft.AspNetCore.Mvc.Testing.WebApplicationFactoryContentRootAttribute">
         <_Parameter1>%(WebApplicationFactoryContentRootAttribute.AssemblyName)</_Parameter1>
@@ -55,5 +55,5 @@
 
     <Copy SourceFiles="%(DepsFilePaths.FullPath)" DestinationFolder="$(OutDir)" Condition="Exists('%(DepsFilePaths.FullPath)')" />
   </Target>
-    
+
 </Project>
\ No newline at end of file
diff --git a/src/ProjectTemplates/BlazorTemplates.Tests/BlazorTemplates.Tests.csproj b/src/ProjectTemplates/BlazorTemplates.Tests/BlazorTemplates.Tests.csproj
index 3118aac607e7..9d082de74f6e 100644
--- a/src/ProjectTemplates/BlazorTemplates.Tests/BlazorTemplates.Tests.csproj
+++ b/src/ProjectTemplates/BlazorTemplates.Tests/BlazorTemplates.Tests.csproj
@@ -69,7 +69,7 @@
     </AssemblyAttribute>
   </ItemGroup>
 
-  <Target Name="PrepareForTest" BeforeTargets="CoreCompile" Condition="$(DesignTimeBuild) != true">
+  <Target Name="PrepareForTest" BeforeTargets="GetAssemblyAttributes" Condition="$(DesignTimeBuild) != true">
     <PropertyGroup>
       <TestTemplateCreationFolder>$([MSBuild]::NormalizePath('$(OutputPath)$(TestTemplateCreationFolder)'))</TestTemplateCreationFolder>
       <TestTemplateTestsProps>$(TestTemplateCreationFolder)$(TestTemplateTestsProps)</TestTemplateTestsProps>
diff --git a/src/ProjectTemplates/test/ProjectTemplates.Tests.csproj b/src/ProjectTemplates/test/ProjectTemplates.Tests.csproj
index e54b80fa4cd5..f9d8b8db4cbc 100644
--- a/src/ProjectTemplates/test/ProjectTemplates.Tests.csproj
+++ b/src/ProjectTemplates/test/ProjectTemplates.Tests.csproj
@@ -72,7 +72,7 @@
     </AssemblyAttribute>
   </ItemGroup>
 
-  <Target Name="PrepareForTest" BeforeTargets="CoreCompile" Condition="$(DesignTimeBuild) != true">
+  <Target Name="PrepareForTest" BeforeTargets="GetAssemblyAttributes" Condition="$(DesignTimeBuild) != true">
     <PropertyGroup>
       <TestTemplateCreationFolder>$([MSBuild]::NormalizePath('$(OutputPath)$(TestTemplateCreationFolder)'))</TestTemplateCreationFolder>
       <TestTemplateTestsProps>$(TestTemplateCreationFolder)$(TestTemplateTestsProps)</TestTemplateTestsProps>
diff --git a/src/Shared/E2ETesting/E2ETesting.targets b/src/Shared/E2ETesting/E2ETesting.targets
index f3e73276b087..5726735dfbf5 100644
--- a/src/Shared/E2ETesting/E2ETesting.targets
+++ b/src/Shared/E2ETesting/E2ETesting.targets
@@ -77,7 +77,7 @@
     </ItemGroup>
   </Target>
 
-  <Target Name="_AddTestProjectMetadataAttributes" BeforeTargets="BeforeCompile" DependsOnTargets="_ResolveTestProjectReferences">
+  <Target Name="_AddTestProjectMetadataAttributes" BeforeTargets="GetAssemblyAttributes" DependsOnTargets="_ResolveTestProjectReferences">
     <ItemGroup>
       <_ContentRootMetadata
         Condition="'%(_ContentRootProjectReferences.Identity)' != ''"
@@ -98,7 +98,7 @@
     </ItemGroup>
   </Target>
 
-  <Target Name="_AddSeleniumSupportedMetadataAttribute" BeforeTargets="BeforeCompile">
+  <Target Name="_AddSeleniumSupportedMetadataAttribute" BeforeTargets="GetAssemblyAttributes">
     <PropertyGroup>
       <_SeleniumE2ETestsSupportedAttributeValue>false</_SeleniumE2ETestsSupportedAttributeValue>
       <_SeleniumE2ETestsSupportedAttributeValue Condition="'$(SeleniumE2ETestsSupported)' == 'true'">true</_SeleniumE2ETestsSupportedAttributeValue>
@@ -118,7 +118,7 @@
     </ItemGroup>
   </Target>
 
-  <Target Name="_AddProcessTrackingMetadataAttribute" BeforeTargets="BeforeCompile">
+  <Target Name="_AddProcessTrackingMetadataAttribute" BeforeTargets="GetAssemblyAttributes">
     <MakeDir Directories="$(SeleniumProcessTrackingFolder)" />
     <ItemGroup>
       <AssemblyAttribute