From d0c2cc058f543e7adf5d20b155787d1fa0b87402 Mon Sep 17 00:00:00 2001
From: Davis Goodin <dagood@users.noreply.github.com>
Date: Fri, 22 Mar 2019 13:54:03 -0500
Subject: [PATCH] Fix WindowsDesktop file list root attributes
 (dotnet/core-setup#5533)

Stop automatically inheriting the .NET Core targeting pack's values.

Commit migrated from https://github.com/dotnet/core-setup/commit/42bdab6aa6ebfb40e9cb99de2542b6fb326bf44f
---
 src/installer/pkg/projects/dir.props                          |  7 -------
 src/installer/pkg/projects/dir.targets                        | 11 +++++++++++
 src/installer/pkg/projects/netcoreapp/pkg/dir.props           |  7 +++++++
 src/installer/pkg/projects/windowsdesktop/dir.props           |  8 ++++++++
 src/installer/pkg/projects/windowsdesktop/pkg/dir.props       |  9 ++++++++-
 .../pkg/projects/windowsdesktop/src/windowsdesktop.depproj    |  4 ----
 6 files changed, 34 insertions(+), 12 deletions(-)
 create mode 100644 src/installer/pkg/projects/windowsdesktop/dir.props

diff --git a/src/installer/pkg/projects/dir.props b/src/installer/pkg/projects/dir.props
index 671e68d..fbfc46b 100644
--- a/src/installer/pkg/projects/dir.props
+++ b/src/installer/pkg/projects/dir.props
@@ -30,13 +30,6 @@
     <ProjectExclusions Include="windowsdesktop\**\*.*proj" />
   </ItemGroup>
 
-  <ItemGroup>
-    <FrameworkListRootAttributes Include="Name" Value="$(NETCoreAppFrameworkBrandName)" />
-    <FrameworkListRootAttributes Include="TargetFrameworkIdentifier" Value="$(NETCoreAppFrameworkIdentifier)" />
-    <FrameworkListRootAttributes Include="TargetFrameworkVersion" Value="$(NETCoreAppFrameworkVersion)" />
-    <FrameworkListRootAttributes Include="FrameworkName" Value="$(SharedFrameworkName)" />
-  </ItemGroup>
-
   <PropertyGroup Condition="'$(PackageTargetRuntime)' == ''">
     <SkipValidatePackage>true</SkipValidatePackage>
     <IncludeRuntimeJson>true</IncludeRuntimeJson>
diff --git a/src/installer/pkg/projects/dir.targets b/src/installer/pkg/projects/dir.targets
index f389726..3bac47a 100644
--- a/src/installer/pkg/projects/dir.targets
+++ b/src/installer/pkg/projects/dir.targets
@@ -193,6 +193,17 @@
       <FrameworkListFile>$(IntermediateOutputPath)FrameworkList.xml</FrameworkListFile>
     </PropertyGroup>
 
+    <ItemGroup>
+      <FrameworkListRootAttributes Include="Name" Value="$(FrameworkListName)" />
+      <FrameworkListRootAttributes Include="TargetFrameworkIdentifier" Value="$(FrameworkListTargetFrameworkIdentifier)" />
+      <FrameworkListRootAttributes Include="TargetFrameworkVersion" Value="$(FrameworkListTargetFrameworkVersion)" />
+      <FrameworkListRootAttributes Include="FrameworkName" Value="$(FrameworkListFrameworkName)" />
+    </ItemGroup>
+
+    <Error
+      Condition="'%(FrameworkListRootAttributes.Value)' == ''"
+      Text="Missing value for property 'FrameworkList%(FrameworkListRootAttributes.Identity)'" />
+
     <CreateFrameworkListFile
       Files="@(File)"
       TargetFile="$(FrameworkListFile)"
diff --git a/src/installer/pkg/projects/netcoreapp/pkg/dir.props b/src/installer/pkg/projects/netcoreapp/pkg/dir.props
index 83db445..6275811 100644
--- a/src/installer/pkg/projects/netcoreapp/pkg/dir.props
+++ b/src/installer/pkg/projects/netcoreapp/pkg/dir.props
@@ -7,6 +7,13 @@
 
   <Import Project="..\..\dir.props" />
 
+  <PropertyGroup>
+    <FrameworkListName>$(NETCoreAppFrameworkBrandName)</FrameworkListName>
+    <FrameworkListTargetFrameworkIdentifier>$(NETCoreAppFrameworkIdentifier)</FrameworkListTargetFrameworkIdentifier>
+    <FrameworkListTargetFrameworkVersion>$(NETCoreAppFrameworkVersion)</FrameworkListTargetFrameworkVersion>
+    <FrameworkListFrameworkName>$(SharedFrameworkName)</FrameworkListFrameworkName>
+  </PropertyGroup>
+
   <!-- Redistribute package content from other nuget packages. -->
   <ItemGroup>
     <ProjectReference Include="..\src\netcoreapp.depproj">
diff --git a/src/installer/pkg/projects/windowsdesktop/dir.props b/src/installer/pkg/projects/windowsdesktop/dir.props
new file mode 100644
index 0000000..6a02ce6
--- /dev/null
+++ b/src/installer/pkg/projects/windowsdesktop/dir.props
@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="utf-8"?>
+<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory).., dir.props))\dir.props" />
+
+  <PropertyGroup>
+    <FrameworkPackageName>Microsoft.WindowsDesktop.App</FrameworkPackageName>
+  </PropertyGroup>
+</Project>
diff --git a/src/installer/pkg/projects/windowsdesktop/pkg/dir.props b/src/installer/pkg/projects/windowsdesktop/pkg/dir.props
index a745bbd..ce92d2e 100644
--- a/src/installer/pkg/projects/windowsdesktop/pkg/dir.props
+++ b/src/installer/pkg/projects/windowsdesktop/pkg/dir.props
@@ -6,7 +6,14 @@
     <ProductBrandPrefix>Microsoft Windows Desktop</ProductBrandPrefix>
   </PropertyGroup>
 
-  <Import Project="..\..\dir.props" />
+  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory).., dir.props))\dir.props" />
+
+  <PropertyGroup>
+    <FrameworkListName>Windows Desktop $(NETCoreAppFrameworkVersion)</FrameworkListName>
+    <FrameworkListTargetFrameworkIdentifier>$(NETCoreAppFrameworkIdentifier)</FrameworkListTargetFrameworkIdentifier>
+    <FrameworkListTargetFrameworkVersion>$(NETCoreAppFrameworkVersion)</FrameworkListTargetFrameworkVersion>
+    <FrameworkListFrameworkName>$(FrameworkPackageName)</FrameworkListFrameworkName>
+  </PropertyGroup>
 
   <PropertyGroup>
     <BuildDebPackage>false</BuildDebPackage>
diff --git a/src/installer/pkg/projects/windowsdesktop/src/windowsdesktop.depproj b/src/installer/pkg/projects/windowsdesktop/src/windowsdesktop.depproj
index 471db17..ca872f8 100644
--- a/src/installer/pkg/projects/windowsdesktop/src/windowsdesktop.depproj
+++ b/src/installer/pkg/projects/windowsdesktop/src/windowsdesktop.depproj
@@ -2,10 +2,6 @@
 <Project ToolsVersion="14.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003" DefaultTargets="Build">
   <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), dir.props))\dir.props" />
 
-  <PropertyGroup>
-    <FrameworkPackageName>Microsoft.WindowsDesktop.App</FrameworkPackageName>
-  </PropertyGroup>
-
   <ItemGroup>
     <PackageReference Include="Microsoft.WindowsDesktop.App">
       <Version>$(MicrosoftWindowsDesktopAppPackageVersion)</Version>
-- 
2.7.4