Use source-build-assets repo - #6044
#6044Use source-build-assets repo#6044
source-build-assets repo#6044Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the VMR to consume the renamed dotnet/source-build-assets repository by replacing source-build-reference-packages / “SBRP” references across build logic, tooling, and documentation.
Changes:
- Renames repo references (paths, repo names, and feed/source names) from SBRP to SBA across VMR build orchestration.
- Updates MSBuild tasks and build props/targets to use the new repo name and source paths.
- Refreshes documentation links and guidance to point to
dotnet/source-build-assets.
Reviewed changes
Copilot reviewed 50 out of 10397 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/source-build-assets/src/externalPackages/projects/Directory.Build.targets | Renames NuGet source/comment from SBRP to SBA within source-build-assets build targets. |
| src/source-build-assets/azure-pipelines/builds/cleanup-unreferenced-packages.yml | Updates pipeline repo name and path trimming to source-build-assets. |
| src/source-build-assets/README.md | Updates repo links/name references to source-build-assets. |
| src/source-build-assets/.devcontainer/init.sh | Updates expected workspace folder name for the repo. |
| src/source-build-assets/.devcontainer/init-toolset.sh | Updates default workspace path to source-build-assets. |
| src/source-build-assets/.config/tsaoptions.json | Updates TSA repo/codebase names to source-build-assets. |
| src/sdk/src/Tasks/Microsoft.NET.Build.Extensions.Tasks/Microsoft.NET.Build.Extensions.Tasks.csproj | Updates comment to reference SBA-produced version. |
| src/sdk/src/Cli/dotnet/dotnet.csproj | Updates comment to reference SBA-produced version. |
| src/sdk/eng/Versions.props | Updates comments/links to new repo for text-only packages guidance. |
| src/sdk/eng/Version.Details.xml | Updates comment references from SBRP to SBA. |
| src/razor/eng/Version.Details.xml | Updates comment references from SBRP to SBA. |
| src/msbuild/eng/dependabot/Directory.Packages.props | Updates comment link to new repo path. |
| src/msbuild/eng/Versions.props | Updates comment guidance to add versions to source-build-assets. |
| src/msbuild/documentation/specs/proposed/interactive-package-references.md | Updates documentation link to externalPackages in source-build-assets. |
| src/arcade/Documentation/update-xunit.md | Updates docs to point to new repo (partially). |
| src/arcade/Documentation/V3StatusUpdate.md | Renames repo row to source-build-assets. |
| src/arcade/Documentation/SBOMGenerationGuidance.md | Updates repo list entry to source-build-assets. |
| repo-projects/vstest.proj | Switches repository reference to source-build-assets. |
| repo-projects/templating.proj | Switches repository reference to source-build-assets. |
| repo-projects/symreader.proj | Switches repository reference to source-build-assets. |
| repo-projects/sourcelink.proj | Switches repository reference to source-build-assets. |
| repo-projects/sdk.proj | Switches repository reference to source-build-assets (and normalizes BOM/format). |
| repo-projects/runtime.proj | Switches repository reference to source-build-assets. |
| repo-projects/roslyn.proj | Switches repository reference to source-build-assets. |
| repo-projects/razor.proj | Switches repository reference to source-build-assets. |
| repo-projects/nuget-client.proj | Switches repository reference to source-build-assets. |
| repo-projects/msbuild.proj | Switches repository reference to source-build-assets. |
| repo-projects/fsharp.proj | Switches repository reference to source-build-assets. |
| repo-projects/emsdk.proj | Switches repository reference to source-build-assets. |
| repo-projects/diagnostics.proj | Switches repository reference to source-build-assets. |
| repo-projects/deployment-tools.proj | Switches repository reference to source-build-assets. |
| repo-projects/command-line-api.proj | Switches repository reference to source-build-assets. |
| repo-projects/cecil.proj | Switches repository reference to source-build-assets. |
| repo-projects/aspnetcore.proj | Switches repository reference to source-build-assets. |
| repo-projects/arcade.proj | Switches repository reference to source-build-assets. |
| repo-projects/Directory.Build.targets | Renames task parameters/metadata from SBRP to SBA and updates repo-name conditions. |
| repo-projects/Directory.Build.props | Updates cache feed value to source-build-assets-cache (property name not updated). |
| eng/tools/tasks/Microsoft.DotNet.UnifiedBuild.Tasks/WriteSbrpUsageReport.cs | Renames internal logic/output to SBA while keeping task/type name. |
| eng/tools/tasks/Microsoft.DotNet.UnifiedBuild.Tasks/WritePackageVersionsProps.cs | Updates comment to reference source-build-assets. |
| eng/tools/tasks/Microsoft.DotNet.UnifiedBuild.Tasks/UpdateNuGetConfigPackageSourcesMappings.cs | Renames task parameters/logic from SBRP cache discovery to SBA. |
| eng/finish-source-only.proj | Updates task invocation arguments to SBA paths. |
| eng/allowed-vmr-binaries.txt | Updates allow-list paths for moved repo directory. |
| eng/allowed-sb-binaries.txt | Updates allow-list paths for moved repo directory. |
| eng/VmrLayout.props | Replaces SbrpRepoSrcDir with SourceBuildAssetsRepoSrcDir. |
| eng/PublishSourceBuild.props | Renames item group for poison checking from SBRP to SBA. |
| docs/VMR-Strategy-For-External-Source.md | Updates strategy doc references/links to source-build-assets. |
| docs/VMR-Design-And-Operation.md | Updates VMR layout docs to src/source-build-assets/. |
| docs/TFM-Trimming-And-Targeting.md | Updates link to new repo. |
| Directory.Build.props | Updates BootstrapArcadeRepos to use source-build-assets. |
| .config/CredScanSuppressions.json | Updates suppression paths to new repo directory. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hmm, more conflicts, as old SBRP path was updated with a new repo flow... Will rebase/resolve offline and force-push. |
It would be good to merge this PR soon, i.e. today. @mthalman @ellahathaway @marcpopMSFT @premun - fyi |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
8792336 to
044a916
Compare
|
@NikolaMilosavljevic I approved so feel free to merge. Someone will have to bypass the merge policies when the build is green again (you might need to JIT for that? I saw @mthalman doing that yesterday so when you are both around, go ahead). |
|
In the meantime, I'd suggest putting blocking reviews on SBRP FFs |
Newer SDK versions contain a repository layout change after source-build-reference-packages repo was renamed to source-build-assets As a result, cherry-picking e64f84d fails with a file-location conflict PR:dotnet/dotnet#6044 This PR resolves the e64f84d backport conflict caused by this migration by automatically staging the relocated patch file and continuing the cherry-pick. This remove manual conflict resolution. Signed-off-by: shreya <raoshreya412@gmail.com>
Newer SDK versions contain a repository layout change after source-build-reference-packages repo was renamed to source-build-assets As a result, cherry-picking e64f84d fails with a file-location conflict PR:dotnet/dotnet#6044 This PR resolves the e64f84d backport conflict caused by this migration by automatically staging the relocated patch file and continuing the cherry-pick. This remove manual conflict resolution. Signed-off-by: shreya <raoshreya412@gmail.com> Signed-off-by: Shreya Rao <raoshreya412@gmail.com>
Newer SDK versions contain a repository layout change after source-build-reference-packages repo was renamed to source-build-assets As a result, cherry-picking e64f84d fails with a file-location conflict PR:dotnet/dotnet#6044 This PR resolves the e64f84d backport conflict caused by this migration by automatically staging the relocated patch file and continuing the cherry-pick. This remove manual conflict resolution. Signed-off-by: shreya <raoshreya412@gmail.com> Signed-off-by: Shreya Rao <raoshreya412@gmail.com>
After
source-build-reference-packagesrepo was renamed tosource-build-assets, changes are needed in VMR to allow proper consumption and flow.This PR updates repo-related properties or values, as well as various other references in code and documentation. If something is not renamed, I'd prefer to prepare a follow up change in a new PR, to not delay this work any longer, considering timelines for next preview release and code-complete date.
Validation
VMR build