From d3fb5e76f1fdf6a581c65316af4170f46d40fe7c Mon Sep 17 00:00:00 2001
From: Nate McMaster <natemcmaster@users.noreply.github.com>
Date: Wed, 9 Aug 2017 16:48:02 -0700
Subject: [PATCH] Remove dependency on a custom task to detect when compiling
 on macOS (#1987)

MSBuild.IsOSPlatform is new to MSBuild 15.3 and can be used to determine when we are building on macOS.
---
 .../Kestrel.FunctionalTests.csproj                   | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/test/Kestrel.FunctionalTests/Kestrel.FunctionalTests.csproj b/test/Kestrel.FunctionalTests/Kestrel.FunctionalTests.csproj
index d1e519263f..845ea0f84c 100644
--- a/test/Kestrel.FunctionalTests/Kestrel.FunctionalTests.csproj
+++ b/test/Kestrel.FunctionalTests/Kestrel.FunctionalTests.csproj
@@ -7,6 +7,7 @@
     <RootNamespace>Microsoft.AspNetCore.Server.Kestrel.FunctionalTests</RootNamespace>
     <TargetFrameworks>netcoreapp2.0;net461</TargetFrameworks>
     <TargetFrameworks Condition=" '$(OS)' != 'Windows_NT' ">netcoreapp2.0</TargetFrameworks>
+    <DefineConstants Condition="$([MSBuild]::IsOSPlatform('OSX'))">$(DefineConstants);MACOS</DefineConstants>
     <ServerGarbageCollection>true</ServerGarbageCollection>
 
     <!--
@@ -42,15 +43,4 @@
     <PackageReference Include="xunit.runner.visualstudio" Version="$(XunitVersion)" />
   </ItemGroup>
 
-  <Target Name="DefinePlatformConstants" BeforeTargets="CoreCompile">
-    <Sdk_GetOSPlatform>
-      <!-- Returns {Linux, macOS, Windows} -->
-      <Output TaskParameter="PlatformName" PropertyName="OSPlatform" />
-    </Sdk_GetOSPlatform>
-
-    <PropertyGroup>
-      <DefineConstants>$(DefineConstants);$(OSPlatform.ToUpperInvariant())</DefineConstants>
-    </PropertyGroup>
-  </Target>
-
 </Project>