Skip to content
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

[main] Revert private assets in deps.json change (#45259) #46324

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

github-actions[bot]
Copy link
Contributor

Backport of #46182 to main

/cc @ViktorHofer @Forgind

This reverts commit 25ad186, reversing
changes made to 152a5d2.
@github-actions github-actions bot requested review from AntonLapounov and a team as code owners January 27, 2025 11:20
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Jan 27, 2025
@ViktorHofer ViktorHofer enabled auto-merge (squash) January 27, 2025 11:24
@ViktorHofer ViktorHofer merged commit 23e2ba8 into main Jan 27, 2025
36 of 38 checks passed
@ViktorHofer ViktorHofer deleted the backport/pr-46182-to-main branch January 27, 2025 13:21
nguerrera added a commit to nguerrera/cask that referenced this pull request Jan 31, 2025
For a period, there was breaking SDK change with `PrivateAssets="all"` stripping things from the build deps.json. It's been [reverted](dotnet/sdk#46324) but that revert hasn't made it to preview builds of VS yet.

While this breaking change is around, having a `PrivateAssets="all"` reference to Microsoft.NET.Test.Sdk causes:

```
Testhost process for source(s) 'D:\Src\cask\bld\bin\Cask.Tests\debug_net8.0\Cask.Tests.dll' exited with error: Unhandled exception.
System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.TestPlatform.CoreUtilities, Version=15.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.`
```

There's no reason real for our Microsoft.NET.Test.Sdk reference to have `PrivateAssets="all"` and removing it works around the issue so this change removes it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants