From 50660f31c8f0b2d6f72da9d94f046141017c0698 Mon Sep 17 00:00:00 2001
From: John Luo <johluo@microsoft.com>
Date: Tue, 27 Aug 2019 11:18:52 -0700
Subject: [PATCH] Remove prebuilts for source build (#13360)

---
 eng/Tools.props         | 4 +++-
 eng/Workarounds.targets | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/eng/Tools.props b/eng/Tools.props
index 46285b1448..cdfb67026c 100644
--- a/eng/Tools.props
+++ b/eng/Tools.props
@@ -6,7 +6,9 @@
       Without this here, we see regular failures with 'error MSB4236: The SDK 'Yarn.MSBuild' specified could not be found.'
       Since this project is evaluated before .npmproj files are loaded, this should cause the package to end
       up in the NuGet cache ahead of time.
+
+      This is not needed in source build.
     -->
-    <PackageReference Include="Yarn.MSBuild" Version="1.15.2" />
+    <PackageReference Condition="'$(DotNetBuildFromSource)' != 'true'" Include="Yarn.MSBuild" Version="1.15.2" />
   </ItemGroup>
 </Project>
diff --git a/eng/Workarounds.targets b/eng/Workarounds.targets
index f5c74cdd43..5da0f1e726 100644
--- a/eng/Workarounds.targets
+++ b/eng/Workarounds.targets
@@ -25,8 +25,8 @@
   <!-- Workaround https://github.com/dotnet/source-build/issues/1112. Source link is currently disabled in source build so define this dummy target which is required for pack. -->
   <Import Condition="'$(DotNetBuildFromSource)' == 'true'" Project="WorkaroundsImported.targets" />
 
-  <!-- Workaround for https://github.com/dotnet/arcade/issues/204 -->
-  <ItemGroup>
+  <!-- Workaround for https://github.com/dotnet/arcade/issues/204, not needed in source build -->
+  <ItemGroup Condition="'$(DotNetBuildFromSource)' != 'true'">
     <PackageReference Include="Internal.AspNetCore.BuildTasks" PrivateAssets="All" Version="$(InternalAspNetCoreBuildTasksPackageVersion)" IsImplicitlyDefined="true" />
   </ItemGroup>