From e039add584070f5ae15a17326e183d9190d61cdb Mon Sep 17 00:00:00 2001
From: Layomi Akinrinade <layomia@gmail.com>
Date: Sun, 9 Apr 2023 10:34:15 -0700
Subject: [PATCH] Simplify target to enable config binding src generator from
 NuGet package, ensure it runs for all TFMs, and rename enabling property
 (#84379)

* Simplify target to enable config binding src generator and rename sentinel property

* Further simplify target

* Address feedback

* Ensure generator removal target is for all TFMs

* Add logic to emit correct recommended minimum TFM
---
 ...fByDefaultRoslynComponent.targets.template | 17 ---------
 eng/packaging.targets                         | 35 -------------------
 ...oft.Extensions.Configuration.Binder.csproj |  6 +++-
 ...ft.Extensions.Configuration.Binder.targets | 26 ++++++++++++++
 4 files changed, 31 insertions(+), 53 deletions(-)
 delete mode 100644 eng/OffByDefaultRoslynComponent.targets.template
 create mode 100644 src/libraries/Microsoft.Extensions.Configuration.Binder/src/buildTransitive/Microsoft.Extensions.Configuration.Binder.targets

diff --git a/eng/OffByDefaultRoslynComponent.targets.template b/eng/OffByDefaultRoslynComponent.targets.template
deleted file mode 100644
index 056fa1a376fa3..0000000000000
--- a/eng/OffByDefaultRoslynComponent.targets.template
+++ /dev/null
@@ -1,17 +0,0 @@
-<Project>
-    <Target Name="_{TargetPrefix}GatherAnalyzers">
-        <ItemGroup>
-            <_{TargetPrefix}Analyzer Include="@(Analyzer)" Condition="'%(Analyzer.NuGetPackageId)' == '{NuGetPackageId}'" />
-        </ItemGroup>
-    </Target>
-
-    <Target Name="_{TargetPrefix}RemoveAnalyzer" 
-            Condition="'$({EnableSourceGeneratorPropertyName})' != 'true'"
-            AfterTargets="ResolvePackageDependenciesForBuild;ResolveNuGetPackageAssets"
-            DependsOnTargets="_{TargetPrefix}GatherAnalyzers">
-
-        <ItemGroup>
-            <Analyzer Remove="@(_{TargetPrefix}Analyzer)" />
-        </ItemGroup>
-    </Target>
-</Project>
diff --git a/eng/packaging.targets b/eng/packaging.targets
index 264fc6828181f..5dbadba029db9 100644
--- a/eng/packaging.targets
+++ b/eng/packaging.targets
@@ -193,41 +193,6 @@
                       Overwrite="true" />
   </Target>
 
-  <PropertyGroup>
-    <_OffByDefaultRoslynComponentTargetsTemplate>$(MSBuildThisFileDirectory)OffByDefaultRoslynComponent.targets.template</_OffByDefaultRoslynComponentTargetsTemplate>
-    <OffByDefaultRoslynComponentTargetsFileIntermediatePath>$(IntermediateOutputPath)OffByDefaultRoslynComponent.targets</OffByDefaultRoslynComponentTargetsFileIntermediatePath>
-    <RoslynComponentsAreOffByDefault Condition="'$(RoslynComponentsAreOffByDefault)' == ''">false</RoslynComponentsAreOffByDefault>
-  </PropertyGroup>
-
-  <!-- In packages that contain Analyzers that are off by default, include a .targets file that will exclude the analyzer. -->
-  <Target Name="ExcludeOffByDefaultRoslynComponentTargetsInPackage"
-          AfterTargets="IncludeAnalyzersInPackage"
-          Condition="'@(ProjectReference)' != '' and
-                     @(ProjectReference->AnyHaveMetadataValue('PackAsAnalyzer', 'true')) and
-                     '$(RoslynComponentsAreOffByDefault)' == 'true'"
-          DependsOnTargets="GenerateOffByDefaultRoslynComponentTargetsFile">
-    <ItemGroup>
-      <Content Include="$(OffByDefaultRoslynComponentTargetsFileIntermediatePath)" PackagePath="buildTransitive\netstandard2.0\$(PackageId).targets" />
-      <Content Include="$(OffByDefaultRoslynComponentTargetsFileIntermediatePath)" PackagePath="buildTransitive\%(NETStandardCompatError.Supported)\$(PackageId).targets" Condition="'@(NETStandardCompatError)' != ''" />
-    </ItemGroup>
-  </Target>
-
-  <Target Name="GenerateOffByDefaultRoslynComponentTargetsFile"
-          Inputs="$(MSBuildProjectFullPath);$(_OffByDefaultRoslynComponentTargetsTemplate)"
-          Outputs="$(OffByDefaultRoslynComponentTargetsFileIntermediatePath)">
-    <PropertyGroup>
-      <_OffByDefaultRoslynComponentTargetPrefix>$(PackageId.Replace('.', '_'))</_OffByDefaultRoslynComponentTargetPrefix>
-      <EnableSourceGeneratorPropertyName Condition="'$(EnableSourceGeneratorPropertyName)' == ''">Enable$(PackageId.Replace('.', ''))SourceGenerator</EnableSourceGeneratorPropertyName>
-    </PropertyGroup>
-
-    <WriteLinesToFile File="$(OffByDefaultRoslynComponentTargetsFileIntermediatePath)"
-                      Lines="$([System.IO.File]::ReadAllText('$(_OffByDefaultRoslynComponentTargetsTemplate)')
-                                                 .Replace('{TargetPrefix}', '$(_OffByDefaultRoslynComponentTargetPrefix)')
-                                                 .Replace('{NuGetPackageId}', '$(PackageId)')
-                                                 .Replace('{EnableSourceGeneratorPropertyName}', '$(EnableSourceGeneratorPropertyName)'))"
-                      Overwrite="true" />
-  </Target>
-
   <!-- Add targets file that marks a .NETStandard applicable tfm as unsupported. -->
   <Target Name="AddNETStandardCompatErrorFileForPackaging"
           Condition="'@(NETStandardCompatError)' != '' and '$(DisableNETStandardCompatErrors)' != 'true'"
diff --git a/src/libraries/Microsoft.Extensions.Configuration.Binder/src/Microsoft.Extensions.Configuration.Binder.csproj b/src/libraries/Microsoft.Extensions.Configuration.Binder/src/Microsoft.Extensions.Configuration.Binder.csproj
index 15775cfaf0f75..4895df3a5e539 100644
--- a/src/libraries/Microsoft.Extensions.Configuration.Binder/src/Microsoft.Extensions.Configuration.Binder.csproj
+++ b/src/libraries/Microsoft.Extensions.Configuration.Binder/src/Microsoft.Extensions.Configuration.Binder.csproj
@@ -37,6 +37,10 @@
 
   <PropertyGroup>
     <IncludeMultiTargetRoslynComponentTargets>false</IncludeMultiTargetRoslynComponentTargets>
-    <RoslynComponentsAreOffByDefault>true</RoslynComponentsAreOffByDefault>
+    <DisableNETStandardCompatErrors>true</DisableNETStandardCompatErrors>
   </PropertyGroup>
+
+  <ItemGroup>
+    <Content Include="buildTransitive\$(PackageId).targets" PackagePath="buildTransitive\netstandard2.0\" />
+  </ItemGroup>
 </Project>
diff --git a/src/libraries/Microsoft.Extensions.Configuration.Binder/src/buildTransitive/Microsoft.Extensions.Configuration.Binder.targets b/src/libraries/Microsoft.Extensions.Configuration.Binder/src/buildTransitive/Microsoft.Extensions.Configuration.Binder.targets
new file mode 100644
index 0000000000000..f091c7a57b23a
--- /dev/null
+++ b/src/libraries/Microsoft.Extensions.Configuration.Binder/src/buildTransitive/Microsoft.Extensions.Configuration.Binder.targets
@@ -0,0 +1,26 @@
+<Project InitialTargets="NETStandardCompatError_Microsoft_Extensions_Configuration_Binder">
+    <Target Name="_Microsoft_Extensions_Configuration_BinderRemoveAnalyzer" 
+            Condition="'$(EnableConfigurationBindingGenerator)' != 'true'"
+            AfterTargets="ResolvePackageDependenciesForBuild;ResolveNuGetPackageAssets">
+
+        <ItemGroup>
+            <Analyzer Remove="@(Analyzer->WithMetadataValue('NuGetPackageId', 'Microsoft.Extensions.Configuration.Binder'))" />
+        </ItemGroup>
+    </Target>
+
+     <Target Name="NETStandardCompatError_Microsoft_Extensions_Configuration_Binder"
+          Condition="'$(SuppressTfmSupportBuildWarnings)' == ''">
+        <PropertyGroup>
+            <_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework
+                Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'netcoreapp2.0')) AND
+                           !$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net6.0'))"
+                >net6.0</_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework>
+            <_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework
+                Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net461')) AND
+                           !$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net462'))"
+                >net462</_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework>
+        </PropertyGroup>
+        <Warning Condition="'$(_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework)' != ''"
+                Text="Microsoft.Extensions.Configuration.Binder doesn't support $(TargetFramework) and has not been tested with it. Consider upgrading your TargetFramework to $(_Microsoft_Extensions_Configuration_Binder_Compatible_TargetFramework) or later. You may also set &lt;SuppressTfmSupportBuildWarnings&gt;true&lt;/SuppressTfmSupportBuildWarnings&gt; in the project file to ignore this warning and attempt to run in this unsupported configuration at your own risk." />
+    </Target>
+</Project>