-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
.NET 9 SDK prints relative path for build artifacts (UX) #38552
Comments
Encountering another issue: it seems
|
This also reproduces on a .NET 8 SDK when passing the Note: without the |
If the behaviour is going to be configurable, then the |
This is what happens when I run
It mentions nothing about the tests being executed and all of them passing. |
Live logger explicitly converts the path to relative here: https://github.com/dotnet/msbuild/blob/299e0514835a1588e6ef21b1da748462dec706b8/src/MSBuild/TerminalLogger/TerminalLogger.cs#L453-L465 |
The relative path thing is by design - we got a lot of user feedback that schedule paths cleared the display. The poor test output is also known - the test team is contributing a much better display that we hope to do in preview 2. |
Personally, my preference is the longer absolute path, because I can copy it elsewhere and it will just work.
@baronfel did you also see my comment about |
Yes, the MSBuild repo has a tracking issue for making the 'primary output' of a build (the thing that's shown on the right hand side of the static build summary for each project) configurable. Right now it's tied to the primary output DLL of a project, but packages are a sensible thing to link as well. |
Linking some items in the MSBuild repo for tracking purposes: |
@tmds if we're tracking all of your requests, should we close this? |
Yes, let's close this one. |
With previous versions of the SDK, the SDK printed an absolute path:
The .NET 9 SDK is printing a relative path:
With the new format, it is no longer possible to copy the path and use it as-is.
Version:
cc @baronfel
The text was updated successfully, but these errors were encountered: