Skip to content

[release/9.0] Use source-build-assets repo#66278

Merged
wtgodbe merged 2 commits into
dotnet:release/9.0from
NikolaMilosavljevic:sbrp.9
Apr 13, 2026
Merged

[release/9.0] Use source-build-assets repo#66278
wtgodbe merged 2 commits into
dotnet:release/9.0from
NikolaMilosavljevic:sbrp.9

Conversation

@NikolaMilosavljevic
Copy link
Copy Markdown
Member

source-build-reference-packages repo was renamed to source-build-assets. To enable VMR/source-build scenarios this reference needs to be updated. This also updates the version as the new repo produced a new package.

Copilot AI review requested due to automatic review settings April 11, 2026 19:53
@github-actions github-actions Bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 11, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the source-build intermediate dependency to reflect the source-build-reference-packagessource-build-assets repo rename and new package output.

Changes:

  • Renames the intermediate dependency package from ...source-build-reference-packages to ...source-build-assets.
  • Updates the dependency URI, SHA, and version to the new repo/package.

Comment thread eng/Version.Details.xml
<Uri>https://github.com/dotnet/source-build-reference-packages</Uri>
<Sha>1b838a42e4952b8fdf212cb1b43c5ce4d69f27b3</Sha>
<SourceBuild RepoName="source-build-reference-packages" ManagedOnly="true" />
<Dependency Name="Microsoft.SourceBuild.Intermediate.source-build-assets" Version="9.0.0-alpha.1.26208.6">
Copy link

Copilot AI Apr 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This new dependency name will cause eng/scripts/CodeCheck.ps1 to fail because it expects a matching version property in eng/Versions.props derived from the dependency name (dots/dashes removed + 'Version'). After this rename, the expected property becomes 'MicrosoftSourceBuildIntermediatesourcebuildassetsVersion', but Versions.props still contains the old '...sourcebuildreferencepackagesVersion' property. Please update Versions.props accordingly (add the new property with this version and remove/rename the old one) so Version.Details.xml and Versions.props stay in sync.

Suggested change
<Dependency Name="Microsoft.SourceBuild.Intermediate.source-build-assets" Version="9.0.0-alpha.1.26208.6">
<Dependency Name="Microsoft.SourceBuild.Intermediate.source-build-reference-packages" Version="9.0.0-alpha.1.26208.6">

Copilot uses AI. Check for mistakes.
@dotnet-policy-service dotnet-policy-service Bot added this to the 9.0.x milestone Apr 11, 2026
@wtgodbe wtgodbe changed the title Use source-build-assets repo [release/9.0] Use source-build-assets repo Apr 12, 2026
@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Apr 12, 2026
Copy link
Copy Markdown
Member

@wtgodbe wtgodbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also update

<MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion>9.0.0-alpha.1.24413.1</MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion>
& any other uses of the property MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion

@NikolaMilosavljevic
Copy link
Copy Markdown
Member Author

Please also update

<MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion>9.0.0-alpha.1.24413.1</MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion>

& any other uses of the property MicrosoftSourceBuildIntermediatesourcebuildreferencepackagesVersion

Fixed with 6066ccd - the only instance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants