search

dotnet / msbuild

The Microsoft Build Engine (MSBuild) is the build platform for .NET and Visual Studio.
https://docs.microsoft.com/visualstudio/msbuild/msbuild
MIT License
5.21k stars 1.35k forks source link

[Bug]: Error output of `dotnet restore` while `/tl` is on does not show errors #9280

Closed premun closed 11 months ago

premun commented 11 months ago

Issue Description

This is what the output looks like when there's a problem during dotnet restore:

D:\repos\arcade-services> dotnet restore

Restore failed in 4,9s

The problem in this case was a duplicate entry in Directory.Packages.props

Steps to Reproduce

  1. Take any repo that is using CPM (https://learn.microsoft.com/en-us/nuget/consume-packages/central-package-management)
  2. Duplicate an entry in the Directory.Packages.props file (or make another NuGet configuration mistake such as forgetting to add an item for a package used in a project)
  3. Call dotnet restore

Expected Behavior

Errors that show without /tl should show with /tl on so that it's possible to fix the problem.

Actual Behavior

No errors are shown.

Analysis

No response

Versions & Configurations

MSBuild version 17.8.0-preview-23418-03+0125fc9fb for .NET 17.8.0.41803

baronfel commented 11 months ago

@premun I think this is a duplicate of https://github.com/dotnet/msbuild/issues/9082 - can you check? if so then we can close this one.

premun commented 11 months ago

Ah, yes, thanks