From 4fdee21a636fca58ba9a3028a33d398f5bfd7178 Mon Sep 17 00:00:00 2001
From: Jeremy Koritzinsky <jekoritz@microsoft.com>
Date: Thu, 15 Apr 2021 08:08:04 -0700
Subject: [PATCH] Pass down AdditionalProperties for NativeProjectReferences.
 Precalculate if the generator is multi-config so the
 CopyNativeProjectBinaries target doesn't need to call into the CMake project
 (#7237)

---
 .../build/Microsoft.DotNet.CMake.Sdk.targets   |  5 ++++-
 .../sdk/ProjectReference.targets               | 18 ++++++++----------
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/src/Microsoft.DotNet.CMake.Sdk/build/Microsoft.DotNet.CMake.Sdk.targets b/src/Microsoft.DotNet.CMake.Sdk/build/Microsoft.DotNet.CMake.Sdk.targets
index 7ae90ea3044..f8b6cc87f79 100644
--- a/src/Microsoft.DotNet.CMake.Sdk/build/Microsoft.DotNet.CMake.Sdk.targets
+++ b/src/Microsoft.DotNet.CMake.Sdk/build/Microsoft.DotNet.CMake.Sdk.targets
@@ -67,7 +67,7 @@
     </PropertyGroup>
   </Target>
 
-  <Target Name="GetOutputPathForProjectReference" DependsOnTargets="GetRootCMakeListsDirectory;ResolveVSGenerator" Returns="$(_CMakeProjectReferenceFilesLocation)">
+  <Target Name="GetOutputPathForProjectReference" DependsOnTargets="GetRootCMakeListsDirectory;ResolveVSGenerator;IsMultiConfigurationGenerator" Returns="@(_CMakeProjectReferenceFilesLocationItem)">
     <PropertyGroup>
       <_NormalizedReferencedCMakeListsDirectory>$([System.IO.Path]::GetDirectoryName($(ReferencedCMakeLists)))</_NormalizedReferencedCMakeListsDirectory>
     </PropertyGroup>
@@ -83,6 +83,9 @@
       <_CMakeProjectReferenceFilesLocation
         Condition="'$(_CMakeMultiConfigurationGenerator)' == 'true'">$([MSBuild]::NormalizePath('$(_CMakeProjectReferenceFilesLocation)','$(Configuration)'))</_CMakeProjectReferenceFilesLocation>
     </PropertyGroup>
+    <ItemGroup>
+      <_CMakeProjectReferenceFilesLocationItem Include="$(_CMakeProjectReferenceFilesLocation)" IsMultiConfigurationGenerator="_CMakeMultiConfigurationGenerator" />
+    </ItemGroup>
   </Target>
 
   <Target Name="_ResolveVSCompilerToolchainForNonVSGenerators"
diff --git a/src/Microsoft.DotNet.CMake.Sdk/sdk/ProjectReference.targets b/src/Microsoft.DotNet.CMake.Sdk/sdk/ProjectReference.targets
index 02265d6be00..5e58ba92f33 100644
--- a/src/Microsoft.DotNet.CMake.Sdk/sdk/ProjectReference.targets
+++ b/src/Microsoft.DotNet.CMake.Sdk/sdk/ProjectReference.targets
@@ -11,15 +11,10 @@
   </Target>
 
   <Target Name="CopyNativeProjectBinaries">
-    <MSBuild Projects="$(ReferencedCMakeProject)"
-      Targets="IsMultiConfigurationGenerator">
-      <Output TaskParameter="TargetOutputs" PropertyName="_IsMultiConfigurationGenerator" />
-    </MSBuild>
-
-    <ItemGroup Condition="'$(_IsMultiConfigurationGenerator)' == 'true'">
+    <ItemGroup Condition="'$(IsMultiConfigurationGenerator)' == 'true'">
       <NativeProjectBinaries Include="$(NativeProjectOutputFolder)\*.*" />
     </ItemGroup>
-    <ItemGroup Condition="'$(_IsMultiConfigurationGenerator)' != 'true'">
+    <ItemGroup Condition="'$(IsMultiConfigurationGenerator)' != 'true'">
 
     <!-- ############################################################### -->
     <!-- The following is required because the single configuration      -->
@@ -122,12 +117,14 @@
 
     <MSBuild Projects="%(NativeProjectReferenceNormalized.CMakeProject)"
       Targets="GetOutputPathForProjectReference"
-      Properties="ReferencedCMakeLists=%(NativeProjectReferenceNormalized.Identity)">
+      Properties="ReferencedCMakeLists=%(NativeProjectReferenceNormalized.Identity);%(NativeProjectReferenceNormalized.AdditionalProperties)">
       <Output TaskParameter="TargetOutputs" ItemName="NativeProjectOutputFoldersToCopy" />
     </MSBuild>
 
     <ItemGroup>
-      <_NativeProjectReferenceToBuild Include="%(NativeProjectReferenceNormalized.CMakeProject)" Condition="'%(NativeProjectReferenceNormalized.BuildNative)' == 'true'" />
+      <_NativeProjectReferenceToBuild Include="%(NativeProjectReferenceNormalized.CMakeProject)"
+                                      Condition="'%(NativeProjectReferenceNormalized.BuildNative)' == 'true'"
+                                      AdditionalProperties="%(NativeProjectReferenceNormalized.AdditionalProperties)" />
     </ItemGroup>
 
     <RemoveDuplicates Inputs="@(_NativeProjectReferenceToBuild)">
@@ -142,7 +139,8 @@
     <MSBuild Projects="$(MSBuildProjectFile)" 
              Targets="CopyNativeProjectBinaries"
              Properties="NativeProjectOutputFolder=%(NativeProjectOutputFoldersToCopy.Identity);
-                         ReferencedCMakeProject=%(NativeProjectOutputFoldersToCopy.MSBuildSourceProjectFile)"
+                         ReferencedCMakeProject=%(NativeProjectOutputFoldersToCopy.MSBuildSourceProjectFile);
+                         IsMultiConfigurationGenerator=%(NativeProjectOutputFoldersToCopy.IsMultiConfigurationGenerator)"
              Condition="'@(NativeProjectReference)' != ''" />
 
   </Target>