From b691fd4ec93d7a5c382fbeb11a07f01d325aa12f Mon Sep 17 00:00:00 2001
From: Jackson Schuster <36744439+jtschuster@users.noreply.github.com>
Date: Wed, 19 Jul 2023 11:58:31 -0700
Subject: [PATCH] Make DNNE build incrementally (#172)

* MSBuild is annoying and this is required to make it build incrementally
---
 src/msbuild/DNNE.targets | 40 +++++++++++++++++++++++++++-------------
 1 file changed, 27 insertions(+), 13 deletions(-)

diff --git a/src/msbuild/DNNE.targets b/src/msbuild/DNNE.targets
index 7b4d4f5..3b13ed9 100644
--- a/src/msbuild/DNNE.targets
+++ b/src/msbuild/DNNE.targets
@@ -54,14 +54,33 @@ DNNE.targets
     <DnneSupportedTFM Condition="!$([MSBuild]::IsOsPlatform('Windows')) AND '$(DnneIsNetFramework)' == 'true'">false</DnneSupportedTFM>
 
     <DnneCompiledToBinPath>$(DnneGeneratedBinPath)/$(DnneNativeExportsBinaryName)$(DnneNativeBinaryExt)</DnneCompiledToBinPath>
+    <DnneGeneratedSourceFileName>$(DnneGeneratedOutputPath)/$(TargetName).g.c</DnneGeneratedSourceFileName>
   </PropertyGroup>
 
   <ItemGroup>
-    <DnneGeneratedSourceFile Include="$(DnneGeneratedOutputPath)/$(TargetName).g.c" />
-    <DnneNativeExportsOutputs Include="$(DnneNativeExportsBinaryPath)$(DnneNativeExportsBinaryName)$(DnneNativeBinaryExt);$(DnneNativeExportsBinaryPath)$(DnneNativeExportsBinaryName).h;$(DnneNativeExportsBinaryPath)dnne.h" />
+    <DnneGeneratedSourceFile Include="$(DnneGeneratedSourceFileName)" />
+
+    <DnneNativeExportsInput Include="$(DnneCompiledToBinPath)">
+      <OutputFileName>$(DnneNativeExportsBinaryName)$(DnneNativeBinaryExt)</OutputFileName>
+    </DnneNativeExportsInput>
+
+    <DnneNativeExportsInput Include="$(DnneGeneratedSourceFileName)" >
+      <OutputFileName>$(DnneNativeExportsBinaryName).h</OutputFileName>
+    </DnneNativeExportsInput>
+
+    <DnneNativeExportsInput Include="$(DnnePlatformSourcePath)/dnne.h" >
+      <OutputFileName>dnne.h</OutputFileName>
+    </DnneNativeExportsInput>
+
+    <!-- Import libs exist only on the Windows platform -->
+    <DnneNativeExportsInput
+        Include="$(DnneGeneratedBinPath)/$(DnneNativeExportsBinaryName).lib"
+        Condition="$([MSBuild]::IsOsPlatform('Windows'))" >
+      <OutputFileName>$(DnneNativeExportsBinaryName).lib</OutputFileName>
+    </DnneNativeExportsInput>
 
     <!-- Add outputs and general glob to help with project cleanup -->
-    <Clean Include="@(DnneNativeExportsOutputs);$(DnneNativeExportsBinaryPath)$(DnneNativeExportsBinaryName).*"/>
+    <Clean Include="@(DnneNativeExportsInput->'$(DnneNativeExportsBinaryPath)%(OutputFileName)');$(DnneNativeExportsBinaryPath)$(DnneNativeExportsBinaryName).*"/>
   </ItemGroup>
 
   <Target
@@ -92,7 +111,7 @@ DNNE.targets
       they flow through project references.
   -->
   <ItemGroup Condition="'$(DnneBuildExports)' == 'true' AND '$(DnneAddGeneratedBinaryToProject)' == 'true'">
-    <None Include="@(DnneNativeExportsOutputs->'%(FullPath)')">
+    <None Include="@(DnneNativeExportsInput->'$(DnneNativeExportsBinaryPath)%(OutputFileName)')">
       <Link>%(Filename)%(Extension)</Link>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
       <Visible>false</Visible>
@@ -112,8 +131,8 @@ DNNE.targets
   <Target
     Name="DnneBuildNativeExports"
     Condition="('$(DesignTimeBuild)' != 'true' OR '$(BuildingProject)' == 'true') AND '$(DnneSupportedTFM)' == 'true' AND '$(DnneBuildExports)' == 'true'"
-    Inputs="@(DnneGeneratedSourceFile)"
-    Outputs="@(DnneNativeExportsOutputs)"
+    Inputs="@(DnneNativeExportsInput)"
+    Outputs="@(DnneNativeExportsInput->'$(DnneNativeExportsBinaryPath)%(OutputFileName)')"
     AfterTargets="DnneGenerateNativeExports"
     DependsOnTargets="ResolvePackageAssets;ResolveFrameworkReferences">
     <Message Text="Building native exports binary from @(DnneGeneratedSourceFile)" Importance="$(DnneMSBuildLogging)" />
@@ -173,14 +192,9 @@ DNNE.targets
         Deploy the official 'dnne.h' header.
     -->
     <Copy
-        SourceFiles="$(DnneCompiledToBinPath);$(__DnneGeneratedSourceFile);$(DnnePlatformSourcePath)/dnne.h"
-        DestinationFiles="@(DnneNativeExportsOutputs)" />
+        SourceFiles="@(DnneNativeExportsInput)"
+        DestinationFiles="@(DnneNativeExportsInput->'$(DnneNativeExportsBinaryPath)%(OutputFileName)')" />
 
-    <!-- Import libs exist only on the Windows platform -->
-    <Copy
-        Condition="$([MSBuild]::IsOsPlatform('Windows'))"
-        SourceFiles="$(DnneGeneratedBinPath)/$(DnneNativeExportsBinaryName).lib"
-        DestinationFolder="$(DnneNativeExportsBinaryPath)" />
   </Target>
 
   <!--