[release/11.0.1xx-preview1] [dotnet] Fix publish BAR manifest step#24577
Conversation
Publishing is currently failing with:
build-tools\create-packs\Directory.Build.targets(188,5):
error MSB4062: The "PushToBuildStorage" task could not be loaded from the assembly C:\Users\cloudtest\.nuget\packages\microsoft.dotnet.build.tasks.feed\11.0.0-beta.26064.107\build\../tools/net/Microsoft.DotNet.Build.Tasks.Feed.dll.
Could not load file or assembly 'System.Runtime, Version=11.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
The system cannot find the file specified. Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.
[build-tools\create-packs\Microsoft.Android.Sdk.proj]
`PushToBuildStorage` is now targeting .NET 11, but this is a step that
is using a stable .NET SDK (currently .NET 10).
For now, let's just use a slightly older version of the
`PushToBuildStorage` task that targets .NET 10.
When .NET 10 Preview 1 has shipped, it will be easy to use the
`UseDotNet` yaml task and fix this properly.
There was a problem hiding this comment.
Pull request overview
Fixes BAR manifest publishing failures caused by PushToBuildStorage moving to a .NET 11-targeting build tasks assembly while CI is still running that step with a .NET 10 SDK.
Changes:
- Introduce a pinned
Microsoft.DotNet.Build.Tasks.Feedpackage version known to targetnet10.0for CI compatibility. - Update the packaging common project to use the pinned
net10.0-compatible tasks/SDK versions for BAR manifest publishing.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/Versions.props | Adds a new property to pin the last net10.0-targeting Microsoft.DotNet.Build.Tasks.Feed version for CI usage. |
| dotnet/package/common.csproj | Switches BAR publishing-related package references to the pinned net10.0-compatible version to avoid task load failures under .NET 10 SDK. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This reverts commit 8cc3c0c.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [CI Build #ece5179] Build passed (Build packages) ✅Pipeline on Agent |
✅ [CI Build #ece5179] Build passed (Detect API changes) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [CI Build #ece5179] Build passed (Build macOS tests) ✅Pipeline on Agent |
💻 [CI Build #ece5179] Tests on macOS X64 - Mac Sonoma (14) passed 💻✅ All tests on macOS X64 - Mac Sonoma (14) passed. Pipeline on Agent |
💻 [CI Build #ece5179] Tests on macOS arm64 - Mac Tahoe (26) passed 💻✅ All tests on macOS arm64 - Mac Tahoe (26) passed. Pipeline on Agent |
💻 [CI Build #ece5179] Tests on macOS arm64 - Mac Sequoia (15) passed 💻✅ All tests on macOS arm64 - Mac Sequoia (15) passed. Pipeline on Agent |
|
This worked here, so merging: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=13204538&view=results |
Publishing is currently failing with:
PushToBuildStorageis now targeting .NET 11, but this is a step that is using a stable .NET SDK (currently .NET 10).For now, let's just use a slightly older version of the
PushToBuildStoragetask that targets .NET 10.When .NET 10 Preview 1 has shipped, it will be easy to use the
UseDotNetyaml task and fix this properly.This was inspired by / copied from dotnet/android#10721.