Microsoft.NET.ILLink.Tasks generating consolidation notice when trimming enabled

[mtaylorfsmb]

Description

I believe this package is being brought in by the .NET 8 SDK. No matter what I do any project multi-targeting .NET 8 with trimming shows that this package needs to be consolidated in the Nuget Package Manager UI for VS 2022. This is on projects that multi-target.

Reproduction Steps

1. Create a class library that multi-targets to net472 and net6.0.
2. Build and confirm the Nuget packages UI has no issues.
3. Update the project to also target net8.0. (this works by itself)
4. Now conditionally add support for trimming for NET 6+

   <PropertyGroup Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net6.0'))">
     <ImplicitUsings>true</ImplicitUsings>
     <Nullable>disable</Nullable>
     <IsTrimmable>true</IsTrimmable>      
   </PropertyGroup>
   

5. Now go back to the Nuget package manager UI and there are 2 packages for ILLink added and a consolidation message for the project (targeting 7.0)

Expected behavior

No consolidation issue for the ILLink package.

Actual behavior

Generates a consolidation warning in the package manager UI.

Regression?

No response

Known Workarounds

Remove trimming from project and the issue goes away.

Configuration

.NET 472, 6 and 8 with trimming
Class library

Other information

No response

[ghost]

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Description

I believe this package is being brought in by the .NET 8 SDK. No matter what I do any project targeting .NET 8 shows that this package needs to be consolidated in the Nuget Package Manager UI for VS 2022. This is on projects that multi-target.

Reproduction Steps

1. Create a class library that multi-targets to net472 and net6.0.
2. Build and confirm the Nuget packages UI has no issues.
3. Update the project to also target net8.0. (this works by itself)
4. Now conditionally add support for trimming for NET 6+

   <PropertyGroup Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net6.0'))">
     <ImplicitUsings>true</ImplicitUsings>
     <Nullable>disable</Nullable>
     <IsTrimmable>true</IsTrimmable>      
   </PropertyGroup>
   

5. Now go back to the Nuget package manager UI and there are 2 packages for ILLink added and a consolidation message for the project (targeting 7.0)

Expected behavior

No consolidation issue for the ILLink package.

Actual behavior

Generates a consolidation warning in the package manager UI.

Regression?

No response

Known Workarounds

Remove trimming from project and the issue goes away.

Configuration

.NET 472, 6 and 8 with trimming
Class library

Other information

No response

Author:	mtaylorfsmb
Assignees:	-
Labels:	area-Infrastructure-libraries, untriaged
Milestone:	-

[ghost]

Tagging subscribers to this area: @agocke, @sbomer, @vitek-karas
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Description

I believe this package is being brought in by the .NET 8 SDK. No matter what I do any project multi-targeting .NET 8 with trimming shows that this package needs to be consolidated in the Nuget Package Manager UI for VS 2022. This is on projects that multi-target.

Reproduction Steps

1. Create a class library that multi-targets to net472 and net6.0.
2. Build and confirm the Nuget packages UI has no issues.
3. Update the project to also target net8.0. (this works by itself)
4. Now conditionally add support for trimming for NET 6+

   <PropertyGroup Condition="$([MSBuild]::IsTargetFrameworkCompatible('$(TargetFramework)', 'net6.0'))">
     <ImplicitUsings>true</ImplicitUsings>
     <Nullable>disable</Nullable>
     <IsTrimmable>true</IsTrimmable>      
   </PropertyGroup>
   

5. Now go back to the Nuget package manager UI and there are 2 packages for ILLink added and a consolidation message for the project (targeting 7.0)

Expected behavior

No consolidation issue for the ILLink package.

Actual behavior

Generates a consolidation warning in the package manager UI.

Regression?

No response

Known Workarounds

Remove trimming from project and the issue goes away.

Configuration

.NET 472, 6 and 8 with trimming
Class library

Other information

No response

Author:	mtaylorfsmb
Assignees:	-
Labels:	untriaged, area-Tools-ILLink
Milestone:	-

[JDA88]

Thank you @mtaylorfsmb!! I could not find the root cause of the consolidation notice and did not corelate it with the <IsTrimmable>true</IsTrimmable>

Setting it to false fixed the issue.

[sbomer]

Thanks for the repro steps @mtaylorfsmb, I was able to see the same consolidation notice, though it wasn't entirely reliable. I only see it for "Manage NuGet Packages For Solution...", not "Manage NuGet Packages..." for a specific project.

As far as I can tell, the package manager UI shows this notice whenever different versions of a package are referenced across a solution - even if the different versions are referenced for different TFMs. For example, if I add the following, I get a consolidation notice for System.Text.Json:

	<ItemGroup>
		<PackageReference Condition="'$(TargetFramework)' == 'net8.0'" Include="System.Text.Json" Version="8.0.0" />
		<PackageReference Condition="'$(TargetFramework)' == 'net6.0'" Include="System.Text.Json" Version="6.0.0" />
	</ItemGroup>

The SDK effectively adds a PackageReference to Microsoft.NET.ILLink.Tasks, where the referenced version depends on the TFM, so it ends up with the same consolidation notice:

Interestingly, the UI is aware that the package is implicitly referenced by an SDK, but still produces a consolidation notice. I also can't select a Version in the UI - if I click that dropdown, it says "Blocked by project", which suggests this has been intentionally blocked for SDK-referenced packages.

I don't think there's anything we can do on our side to fix this - this would need to be handled in NuGet or VS, if it's not intended behavior.

[sbomer]

Moved to NuGet/Home#13676