From d74f04680f94b22982103cdddd9f6e7e76783d47 Mon Sep 17 00:00:00 2001
From: John Luo <johluo@microsoft.com>
Date: Tue, 30 Apr 2019 15:04:11 -0700
Subject: [PATCH] Restrict workaround to MSBuild 15.0 and above

---
 .../build/_protobuf/Google.Protobuf.Tools.targets        | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/csharp/Grpc.Tools/build/_protobuf/Google.Protobuf.Tools.targets b/src/csharp/Grpc.Tools/build/_protobuf/Google.Protobuf.Tools.targets
index 784f789528e..b3fd02d4faa 100644
--- a/src/csharp/Grpc.Tools/build/_protobuf/Google.Protobuf.Tools.targets
+++ b/src/csharp/Grpc.Tools/build/_protobuf/Google.Protobuf.Tools.targets
@@ -40,10 +40,11 @@
   </ItemGroup>
 
   <!-- Workaround for VS Project System bug: -->
-  <!-- Duplicated items in None itemgroup without Generator attribute prevents the items -->
-  <!-- in Protobuf itemgroup with Generator attribute from triggering design time build. -->
-  <ItemGroup Condition=" '$(DisableProtobufDesignTimeBuild)' != 'true' " >
-    <!-- Working around this by removing the duplicated items from None itemgroup. -->
+  <!-- Duplicated items in None itemgroup without Generator attribute prevents the items in Protobuf -->
+  <!-- itemgroup with Generator attribute from triggering design time build. Working around this by -->
+  <!-- removing the duplicated items from None itemgroup. The Remove attribute was introduced -->
+  <!-- in the .NET Framework 3.5, but was only supported inside targets until MSBuild 15.0. -->
+  <ItemGroup Condition=" '$(MSBuildVersion)' &gt;= '15.0' and '$(DisableProtobufDesignTimeBuild)' != 'true' " >
     <None Remove="@(Protobuf)" />
   </ItemGroup>