Skip to content

Commit

Permalink
Set SignAssembly=false when merging assemblies
Browse files Browse the repository at this point in the history
Since it's unsupported by ilrepack as a dotnet tool (and additionally, entirely unnecessary for analyzers).
  • Loading branch information
kzu authored Jul 9, 2024
1 parent 475ba52 commit 195ed4c
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions samples/dotnet/SponsorLink.Analyzer.targets
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,10 @@
<FundingPrefix Condition="'$(FundingPrefix)' == ''">$([System.Text.RegularExpressions.Regex]::Replace("$(FundingProduct)", "[^A-Z]", ""))</FundingPrefix>
<!-- Default grace days for an expired sponsor manifest or unknown status -->
<FundingGrace Condition="'$(FundingGrace)' == ''">15</FundingGrace>
</PropertyGroup>

<!-- Strong-naming analyzers is both unnecessary and additionally unsupported by ILRepack when run as a dotnet tool -->
<SignAssembly Condition="'$(MergeAnalyzerAssemblies)' == 'true'">false</SignAssembly>
</PropertyGroup>

<ItemGroup>
<Compile Include="$(MSBuildThisFileDirectory)SponsorLink/*.cs"
Expand Down Expand Up @@ -217,4 +220,4 @@ partial class SponsorLink
<SponsorLinkImported>true</SponsorLinkImported>
</PropertyGroup>

</Project>
</Project>

0 comments on commit 195ed4c

Please sign in to comment.