From a278a2624861fc0f464995ed3aa535c6db7f2731 Mon Sep 17 00:00:00 2001
From: Viktor Hofer <viktor.hofer@microsoft.com>
Date: Mon, 22 Jan 2024 18:41:03 +0100
Subject: [PATCH] Make SourceBuild.props work on Windows (#53523)

* Make SourceBuild.props work on Windows

Contributes to VMR work: https://github.com/dotnet/source-build/issues/3926

Changes are from https://github.com/dotnet/dotnet/pull/46

* Update build.ps1
---
 eng/SourceBuild.props | 23 ++++++++++++++++-------
 eng/build.ps1         | 28 +++++++++++++++++-----------
 2 files changed, 33 insertions(+), 18 deletions(-)

diff --git a/eng/SourceBuild.props b/eng/SourceBuild.props
index 93dbe4b4e352..991a0e2e5196 100644
--- a/eng/SourceBuild.props
+++ b/eng/SourceBuild.props
@@ -8,9 +8,11 @@
     <SourceBuildTargetFrameworkFilter>netstandard2.0%3bnetstandard2.1%3bnetcoreapp2.1%3bnetcoreapp3.1%3bnet5.0%3bnet6.0%3bnet7.0%3bnet8.0%3bnet9.0</SourceBuildTargetFrameworkFilter>
   </PropertyGroup>
 
+  <!-- Only run this target in source-build only mode. -->
   <Target Name="PrepareGlobalJsonForSourceBuild"
           AfterTargets="PrepareInnerSourceBuildRepoRoot"
-          BeforeTargets="RunInnerSourceBuildCommand">
+          BeforeTargets="RunInnerSourceBuildCommand"
+          Condition="'$(ArcadeBuildFromSource)' == 'true'">
     <Exec
       Command="./eng/scripts/prepare-sourcebuild-globaljson.sh"
       WorkingDirectory="$(InnerSourceBuildRepoRoot)" />
@@ -28,13 +30,11 @@
     <Delete Files="$(InnerSourceBuildRepoRoot).globalconfig" />
   </Target>
 
-  <!--
-    Build RepoTasks - this is normally triggered via the build script but the inner ArPow source-build is run via msbuild
-  -->
+  <!-- Build RepoTasks - this is normally triggered via the build script but the inner ArPow source-build is run via msbuild.
+       https://github.com/dotnet/source-build/issues/3807 -->
   <Target Name="BuildRepoTasks"
           Condition="'$(ArcadeInnerBuildFromSource)' == 'true'"
           BeforeTargets="Execute">
-
     <!-- If the alternative runtime location and key are present, pass those through -->
     <PropertyGroup>
       <_AdditionalRepoTaskBuildArgs />
@@ -45,7 +45,8 @@
     <ItemGroup>
       <!-- We need to flow FullAssemblySigningSupported even when building repo tasks because they use full signing -->
       <InnerBuildEnv Condition="'$(FullAssemblySigningSupported)' != ''" Include="FullAssemblySigningSupported=$(FullAssemblySigningSupported)" />
-      <InnerBuildEnv Include="DotNetBuildFromSource=true" />
+      <InnerBuildEnv Include="DotNetBuildFromSource=$(ArcadeBuildFromSource)" />
+      <InnerBuildEnv Include="DotNetBuildVertical=$(ArcadeBuildVertical)" />
     </ItemGroup>
 
     <!-- Call the build.sh script to build the repo tasks. Set IgnoreStandardErrorWarningFormat
@@ -56,7 +57,15 @@
       Command="./eng/build.sh --only-build-repo-tasks -bl $(_AdditionalRepoTaskBuildArgs)"
       IgnoreStandardErrorWarningFormat="true"
       WorkingDirectory="$(InnerSourceBuildRepoRoot)"
-      EnvironmentVariables="@(InnerBuildEnv)" />
+      EnvironmentVariables="@(InnerBuildEnv)"
+      Condition="'$(OS)' != 'Windows_NT'" />
+
+    <Exec
+      Command=".\eng\build.cmd -OnlyBuildRepoTasks -bl $(_AdditionalRepoTaskBuildArgs)"
+      IgnoreStandardErrorWarningFormat="true"
+      WorkingDirectory="$(InnerSourceBuildRepoRoot)"
+      EnvironmentVariables="@(InnerBuildEnv)"
+      Condition="'$(OS)' == 'Windows_NT'" />
   </Target>
 
   <Target Name="CustomizeInnerBuildArgs"
diff --git a/eng/build.ps1 b/eng/build.ps1
index 9eca32052315..87389a0f66de 100644
--- a/eng/build.ps1
+++ b/eng/build.ps1
@@ -27,6 +27,9 @@ Do not build project-to-project references and only build the specified project.
 .PARAMETER NoBuildRepoTasks
 Skip building eng/tools/RepoTasks/
 
+.PARAMETER OnlyBuildRepoTasks
+Only build eng/tools/RepoTasks/ and nothing else.
+
 .PARAMETER Pack
 Produce packages.
 
@@ -159,6 +162,7 @@ param(
     [switch]$NoBuildInstallers,
 
     [switch]$NoBuildRepoTasks,
+    [switch]$OnlyBuildRepoTasks,
 
     # Diagnostics
     [Alias('bl')]
@@ -437,20 +441,22 @@ try {
             @ToolsetBuildArguments
     }
 
-    if ($performDesktopBuild) {
-        Write-Host
-        Remove-Item variable:global:_BuildTool -ErrorAction Ignore
-        $msbuildEngine = 'vs'
+    if (-not $OnlyBuildRepoTasks) {
+        if ($performDesktopBuild) {
+            Write-Host
+            Remove-Item variable:global:_BuildTool -ErrorAction Ignore
+            $msbuildEngine = 'vs'
 
-        MSBuild $toolsetBuildProj /p:RepoRoot=$RepoRoot @MSBuildArguments
-    }
+            MSBuild $toolsetBuildProj /p:RepoRoot=$RepoRoot @MSBuildArguments
+        }
 
-    if ($performDotnetBuild) {
-        Write-Host
-        Remove-Item variable:global:_BuildTool -ErrorAction Ignore
-        $msbuildEngine = 'dotnet'
+        if ($performDotnetBuild) {
+            Write-Host
+            Remove-Item variable:global:_BuildTool -ErrorAction Ignore
+            $msbuildEngine = 'dotnet'
 
-        MSBuild $toolsetBuildProj /p:RepoRoot=$RepoRoot @dotnetBuildArguments
+            MSBuild $toolsetBuildProj /p:RepoRoot=$RepoRoot @dotnetBuildArguments
+        }
     }
 }
 catch {