From 4601643ef877dee9d4253d54dfabca5d8b744f7d Mon Sep 17 00:00:00 2001
From: Theodore Tsirpanis <12659251+teo-tsirpanis@users.noreply.github.com>
Date: Tue, 13 Oct 2020 01:18:00 +0300
Subject: [PATCH] Use TargetFramework name to determine the kind of framework.
 (#10247)

Co-authored-by: Saul Rennison <saul@users.noreply.github.com>
---
 FSharp.Profiles.props | 43 ++++++++++++++++++++++++-------------------
 1 file changed, 24 insertions(+), 19 deletions(-)

diff --git a/FSharp.Profiles.props b/FSharp.Profiles.props
index 1abb6c8a9..993512bde 100644
--- a/FSharp.Profiles.props
+++ b/FSharp.Profiles.props
@@ -2,24 +2,29 @@
 <!-- Copyright (c) Microsoft Corporation.  All Rights Reserved.  See License.txt in the project root for license information. -->
 <Project>
 
-  <PropertyGroup Condition="$(TargetFramework.StartsWith('net4'))">
-    <DefineConstants Condition="'$(MonoPackaging)' == 'true'">$(DefineConstants);CROSS_PLATFORM_COMPILER</DefineConstants>
-    <DefineConstants>$(DefineConstants);ENABLE_MONO_SUPPORT</DefineConstants>
-  </PropertyGroup>
-
-  <PropertyGroup Condition="$(TargetFramework.StartsWith('netstandard')) OR $(TargetFramework.StartsWith('netcoreapp'))">
-    <DefineConstants>$(DefineConstants);NETSTANDARD</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_APP_DOMAINS</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_CORHOST_SIGNER</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_PDB_READER</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_PDB_WRITER</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_SYMBOLSTORE</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_SYSTEM_CONFIGURATION</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_WIN_REGISTRY</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_WINFORMS</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_NO_INDENTED_TEXT_WRITER</DefineConstants>
-    <DefineConstants>$(DefineConstants);FX_RESHAPED_REFEMIT</DefineConstants>
-    <OtherFlags>$(OtherFlags) --simpleresolution</OtherFlags>
-  </PropertyGroup>
+  <Choose>
+    <When Condition=" '$(TargetFrameworkIdentifier)' == '.NETFramework' ">
+      <PropertyGroup>
+        <DefineConstants Condition="'$(MonoPackaging)' == 'true'">$(DefineConstants);CROSS_PLATFORM_COMPILER</DefineConstants>
+        <DefineConstants>$(DefineConstants);ENABLE_MONO_SUPPORT</DefineConstants>
+      </PropertyGroup>
+    </When>
+    <Otherwise>
+      <PropertyGroup>
+        <DefineConstants>$(DefineConstants);NETSTANDARD</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_APP_DOMAINS</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_CORHOST_SIGNER</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_PDB_READER</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_PDB_WRITER</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_SYMBOLSTORE</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_SYSTEM_CONFIGURATION</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_WIN_REGISTRY</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_WINFORMS</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_NO_INDENTED_TEXT_WRITER</DefineConstants>
+        <DefineConstants>$(DefineConstants);FX_RESHAPED_REFEMIT</DefineConstants>
+        <OtherFlags>$(OtherFlags) --simpleresolution</OtherFlags>
+      </PropertyGroup>
+    </Otherwise>
+  </Choose>
 
 </Project>