From 2073d1bbc2d319f357fed3bcc9ed0f8ae63def46 Mon Sep 17 00:00:00 2001
From: Omair Majid <omajid@redhat.com>
Date: Sat, 25 Mar 2023 18:39:34 -0400
Subject: [PATCH] Flow FullAssemblySigningSupported for source-build to repo
 tasks

This property is being added to arcade via
https://github.com/dotnet/arcade/pull/12749 and
https://github.com/dotnet/arcade/pull/12940

Once this property is added to arcade, it flows correctly to the main
aspnetcore build, but not to the build for repo tasks. The repo tasks
still need this, otherwise they end up using full signing.

Fix that by manually passing the property along (using env var) when
building the repo tasks.
---
 eng/SourceBuild.props | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/eng/SourceBuild.props b/eng/SourceBuild.props
index 82b297b51f67..59feaabd07b4 100644
--- a/eng/SourceBuild.props
+++ b/eng/SourceBuild.props
@@ -40,6 +40,11 @@
       <_AdditionalRepoTaskBuildArgs Condition="'$(DotNetRuntimeSourceFeedKey)' != ''" >$(_AdditionalRepoTaskBuildArgs) --runtimesourcefeedkey $(DotNetRuntimeSourceFeedKey)</_AdditionalRepoTaskBuildArgs>
     </PropertyGroup>
 
+    <ItemGroup>
+      <!-- We need to flow FullAssemblySigningSupported even when building repo tasks because they use full signing -->
+      <InnerBuildEnv Condition="'$(FullAssemblySigningSupported)' != ''" Include="FullAssemblySigningSupported=$(FullAssemblySigningSupported)" />
+    </ItemGroup>
+
     <!-- Call the build.sh script to build the repo tasks. Set IgnoreStandardErrorWarningFormat
          to true. This avoids fatal errors, because in internal builds there are usually a few failed installation
          attempts as the install script walks through potential locations for a runtime.