Skip to content

Clarify feature band source-build inputs - #5630

Merged
mthalman merged 4 commits into
dotnet:mainfrom
mthalman:mthalman-clarify-feature-band-docs
Aug 13, 2026
Merged

Clarify feature band source-build inputs#5630
mthalman merged 4 commits into
dotnet:mainfrom
mthalman:mthalman-clarify-feature-band-docs

Conversation

@mthalman

@mthalman mthalman commented Aug 5, 2026

Copy link
Copy Markdown
Member

Higher feature-band source builds can fail when inputs are selected by recency rather than by the versions configured in the VMR revision being built. This clarifies how distro maintainers should interpret lifecycle-relative "current" and "previous" inputs while reproducing a specific branch, tag, or commit. It also provides guidance on cross-build scenarios.

Document how lifecycle-relative inputs map to the exact versions configured by a VMR revision, and clarify host versus target inputs for cross-builds.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3746ac39-bec0-467f-8aaf-7ed30f38df0b
@mthalman
mthalman marked this pull request as ready for review August 5, 2026 17:19
@mthalman
mthalman requested a review from a team as a code owner August 5, 2026 17:19
Comment on lines +283 to +284
- **Initial Release (N.0.200)**: Current source-built 1xx shared component
artifacts + previous source-built 1xx artifacts and SDK

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The 2xx section and the 3xx/4xx sections are very similar. It seems like the 2xx sections 3xx/4xx could be combined into a 2xx+ section and use the text from the 3xx/4xx section. Is my understanding correct?

Feel free to dismiss if this is outside the scope of these changes.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in cc280da

`build.sh` script and pass the extracted artifact directory for the
required 1xx version.

**Error**: Package-source mapping or package-version resolution failures after

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: other troubleshooting entries use specific error text here whereas this one doesn't look like a specific error.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in 6751ffa

Comment on lines +821 to +822
components caused `Microsoft.NETCore.Platforms` resolution to fail. That
failure was an unsupported input matrix, not a general 2xx defect.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
components caused `Microsoft.NETCore.Platforms` resolution to fail. That
failure was an unsupported input matrix, not a general 2xx defect.
components causes `Microsoft.NETCore.Platforms` resolution to fail because
the inputs did not match the versions expected by the build.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in b30876e

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3746ac39-bec0-467f-8aaf-7ed30f38df0b
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3746ac39-bec0-467f-8aaf-7ed30f38df0b
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 3746ac39-bec0-467f-8aaf-7ed30f38df0b
@mthalman
mthalman requested a review from lbussell August 12, 2026 20:30
@mthalman
mthalman merged commit 5c1226d into dotnet:main Aug 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants