Build orchestration fixups (#2507)

* Include Microsoft.DotNet.Wpf.DncEng only when running CppHelper in RepoLocation=External. When RepoLocation=Internal, this package is not really needed.

* Document FilterItem1ByItem2 task

* Use /Z7 (i.e., oldStyle) when building dotnet-wpf-int repo. This is needed to workaround dotnet/arcade#4763

* Add missing '\'; improve a condition

* Add `<UndefineProperties>TargetFramework;TargetFrameworks</UndefineProperties>` to native projects. This is needed to prevent these projects from building multiple times parallel.

Author: vatsan-madhavan

eng/WpfArcadeSdk/tools/InlineTasks.targets

@@ -7,6 +7,27 @@
     <TaskFactoryAssemblyToUse Condition="'$(MSBuildProjectExtension)'=='.vcxproj'">$(MSBuildToolsPath)\Microsoft.Build.Tasks.v4.0.dll</TaskFactoryAssemblyToUse>
   </PropertyGroup>
 
+  <!-- 
+    Summary:
+      Selects @(Item2) items that are present in @(Item1). In effect, calculates 
+      set subtraction:
+      
+              Result = Item2 - Item1
+
+    Parameters:
+      @(Item1)                ITaskItem[]   Required
+      PreserveItem1Metadata   Bool          Optional    Preserves item-metadata from @(Item1) in @(Result)
+      
+      @(Item2)                ITaskItem[]   Required
+      Metadata2               String        Required
+      PreserveItem2Metadata   Bool          Optional    Preserves item-metadata from @(Item2) in @(Result)
+      
+      TreatItemsAsPaths       Bool          Optional
+      
+    Outputs:
+      @(Result)               ITaskItem[]
+      @(UnmatchedReferences)  ITaskItem[]
+  --> 
   <UsingTask TaskName="FilterItem1ByItem2"
                TaskFactory="$(TaskFactoryToUse)"
                AssemblyFile="$(TaskFactoryAssemblyToUse)">

eng/WpfArcadeSdk/tools/Wpf.Cpp.props

@@ -208,6 +208,14 @@
 
       <DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
 
+      <!-- 
+        Workaround for https://github.com/dotnet/arcade/issues/4763
+        Embed PDB's into the .lib files directly using /Z7 when building out of 
+        dotnet-wpf-int repo
+      -->
+      <DebugInformationFormat Condition="'$(ManagedCxx)' != 'true' and 
+                                         '$(RepoLocation)' == 'Internal'">oldStyle</DebugInformationFormat>
+      
       <!-- 
         Disambiguate the PDB produced by the compiler with a -compile suffix, so that the PDB
         produced by the linker is not accidentally clobbered by this one.

eng/WpfArcadeSdk/tools/Wpf.Cpp.targets

@@ -462,10 +462,15 @@ using namespace System::Runtime::Versioning;
    The main purpose of this target is to restore the package with PathProperty=true. As a side-effect, 
    we will also generate an Item @(NuGetReferencesForCpp) from IdentifyNuGetReferences target, which
    will be used by ResolveMicrosoftDotNetWpfDncEngReferencesForCpp
+
+   Note: 
+    Test for "$(RepoLocation) != Internal" rather than "$(RepoLocation) == External"
+    The former form is more reliable. 
+
   -->
   <Target Name="CppHelper"
           Returns="@(NuGetReferencesForCpp)"
-          Condition="'$(ManagedCxx)' != 'true' and '$(RepoLocation)' == 'External'">
+          Condition="'$(ManagedCxx)' != 'true' and '$(RepoLocation)' != 'Internal'">
 
     <PropertyGroup>
       <CppHelperProject>$(IntermediateOutputPath)CppSupport\CppSupport_$(MSBuildProjectName).csproj</CppHelperProject>
@@ -499,8 +504,9 @@ using namespace System::Runtime::Versioning;
           <![CDATA[
    <ItemGroup>
       <PackageReference Include="runtime.$(WpfRuntimeIdentifier).Microsoft.DotNet.Wpf.DncEng"
-                      Version="$(MicrosoftDotNetWpfDncEngVersion)" 
-                      GeneratePathProperty="true" />
+                        Condition="'$(MicrosoftDotNetWpfDncEngVersion)' != ''"
+                        Version="$(MicrosoftDotNetWpfDncEngVersion)" 
+                        GeneratePathProperty="true" />
   </ItemGroup>
   ]]>
         </Text>
@@ -521,7 +527,7 @@ using namespace System::Runtime::Versioning;
 
    <PropertyGroup>
       <WpfDncEngRidSpecificLibPath Condition="'%24(RuntimeIdentifier)'=='win-x86' Or '%24(WpfRuntimeIdentifier)'=='win-x86'">%24(Pkgruntime_win-x86_Microsoft_DotNet_Wpf_DncEng)\runtimes\%24(RuntimeIdentifier)\native\</WpfDncEngRidSpecificLibPath>
-      <WpfDncEngRidSpecificLibPath Condition="'%24(RuntimeIdentifier)'=='win-x64' Or '%24(WpfRuntimeIdentifier)'=='win-x64'">%24(Pkgruntime_win-x64_Microsoft_DotNet_Wpf_DncEng)runtimes\%24(RuntimeIdentifier)\native\</WpfDncEngRidSpecificLibPath>
+      <WpfDncEngRidSpecificLibPath Condition="'%24(RuntimeIdentifier)'=='win-x64' Or '%24(WpfRuntimeIdentifier)'=='win-x64'">%24(Pkgruntime_win-x64_Microsoft_DotNet_Wpf_DncEng)\runtimes\%24(RuntimeIdentifier)\native\</WpfDncEngRidSpecificLibPath>
    </PropertyGroup>
 <!-- Save Microsoft.NETCore.App assemblies and return them -->
     <ItemGroup>
@@ -564,7 +570,7 @@ using namespace System::Runtime::Versioning;
     <MSBuild Projects="@(ProjectToBuild)"
          Targets="IdentifyNuGetReferences"
          Properties="TargetFramework=$(TfmForBuild);Platform=$(Architecture);RuntimeIdentifier=win-$(Architecture)">
-      <Output TaskParameter="TargetOutputs" ItemName="NuGetReferencesForCpp" />
+      <Output  TaskParameter="TargetOutputs" ItemName="NuGetReferencesForCpp" />
     </MSBuild>
 
     <!-- Update with file path -->

eng/WpfArcadeSdk/tools/WpfProjectReference.targets

@@ -58,22 +58,26 @@
       <ReferenceOutputAssembly>false</ReferenceOutputAssembly>
       <OutputItemType>Content</OutputItemType>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+      <UndefineProperties>TargetFramework;TargetFrameworks</UndefineProperties>
     </WpfProjectPath>
     <WpfProjectPath Include="WpfGfx_cor3" ProjectPath="$(WpfSourceDir)WpfGfx\core\dll\WpfGfx.vcxproj">
       <ReferenceOutputAssembly>false</ReferenceOutputAssembly>
       <OutputItemType>Content</OutputItemType>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+      <UndefineProperties>TargetFramework;TargetFrameworks</UndefineProperties>    
     </WpfProjectPath>
     <WpfProjectPath Include="PenImc_cor3" ProjectPath="$(WpfSourceDir)PenImc\dll\PenImc.vcxproj">
       <ReferenceOutputAssembly>false</ReferenceOutputAssembly>
       <OutputItemType>Content</OutputItemType>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+      <UndefineProperties>TargetFramework;TargetFrameworks</UndefineProperties>
     </WpfProjectPath>
 
     <WpfProjectPath Include="bilinearspan" ProjectPath="$(WpfSourceDir)WpfGfx\core\sw\bilinearspan\bilinearspan.vcxproj">
       <ReferenceOutputAssembly>false</ReferenceOutputAssembly>
       <OutputItemType>Content</OutputItemType>
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+      <UndefineProperties>TargetFramework;TargetFrameworks</UndefineProperties>
     </WpfProjectPath>
 
   </ItemGroup>