Skip to content

Commit 9dfc64f

Browse files
committed
Fix bug in ILRepack that caused wrong packing
1 parent 5662fb2 commit 9dfc64f

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/Directory.targets

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,16 @@
1616
<PackageFile Include="*.props;*.targets" PackagePath="build\$(TargetFramework)\%(Filename)%(Extension)" Visible="true" />
1717
</ItemGroup>
1818

19-
<Target Name="PackCopyLocalLockFileAssemblies" Condition="'$(IsAnalyzer)' == 'true' and '$(MergeAnalyzerAssemblies)' != 'true'" BeforeTargets="GetPackageContents" DependsOnTargets="ReferenceCopyLocalPathsOutputGroup">
19+
<Target Name="PackCopyLocalLockFileAssemblies" Condition="'$(IsAnalyzer)' == 'true' and '$(ILRepack)' != 'true'" BeforeTargets="GetPackageContents" DependsOnTargets="ReferenceCopyLocalPathsOutputGroup">
20+
<PropertyGroup>
21+
<ILRepackExclude>;@(ILRepackExclude, ';');</ILRepackExclude>
22+
</PropertyGroup>
2023
<ItemGroup>
2124
<ReferenceCopyLocalAssemblies Include="@(ReferenceCopyLocalPaths)" Condition="'%(Extension)' == '.dll'
2225
And !$([MSBuild]::ValueOrDefault('%(FileName)', '').EndsWith('.resources', StringComparison.OrdinalIgnoreCase))
2326
And !$([MSBuild]::ValueOrDefault('%(FileName)', '').StartsWith('ThisAssembly', StringComparison.OrdinalIgnoreCase))" />
2427
<PackCopyLocalAssemblies Include="@(ReferenceCopyLocalAssemblies)" Condition="
28+
!$(ILRepackExclude.Contains(';%(FileName);')) And
2529
!$([MSBuild]::ValueOrDefault('%(FileName)', '').StartsWith('Microsoft.CodeAnalysis', StringComparison.OrdinalIgnoreCase)) And
2630
!$([MSBuild]::ValueOrDefault('%(FileName)', '').StartsWith('Microsoft.CSharp', StringComparison.OrdinalIgnoreCase)) And
2731
!$([MSBuild]::ValueOrDefault('%(FileName)', '').StartsWith('System.', StringComparison.OrdinalIgnoreCase))"

src/ILRepack.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
<ReferenceCopyLocalAssemblies Include="@(ReferenceCopyLocalPaths)" Condition="'%(Extension)' == '.dll'
4444
And !$([MSBuild]::ValueOrDefault('%(FileName)', '').EndsWith('.resources', StringComparison.OrdinalIgnoreCase))" />
4545

46-
<!-- Default to never mergingmerging analyzer, C# or system assemblies -->
46+
<!-- Default to never merging analyzer, C# or system assemblies -->
4747
<MergedAssemblies Include="@(ReferenceCopyLocalAssemblies)" Condition="
4848
!$(ILRepackExclude.Contains(';%(FileName);')) And
4949
!$([MSBuild]::ValueOrDefault('%(FileName)', '').StartsWith('Microsoft.CodeAnalysis', StringComparison.OrdinalIgnoreCase)) And

0 commit comments

Comments
 (0)