Skip to content

[vs18.9] Serialize BuildRequestConfiguration.RequestedTargets to fix solution metaproject MSB4057 in parallel builds - #14227

Merged
ViktorHofer merged 4 commits into
vs18.9from
backport/pr-14223-to-vs18.9
Jul 1, 2026
Merged

ViktorHofer merged 4 commits into
vs18.9from
backport/pr-14223-to-vs18.9

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Backport of #14223 to vs18.9

/cc @ViktorHofer

ViktorHofer and others added 3 commits July 1, 2026 06:51
In parallel/multithreaded builds a BuildRequestConfiguration is round-tripped through translation. RequestedTargets was a get-only auto-property that was never serialized, so the deserialized configuration reset it to an empty collection.

SolutionProjectGenerator emits user-requested targets (e.g. Pack) into the generated .slnx.metaproj only when they are present in config.RequestedTargets. Losing them caused solution builds to fail with MSB4057 (target does not exist) under parallel/MT builds while serial builds worked.

Back RequestedTargets with a serialized field and translate it in BuildRequestConfiguration.Translate. Adds a regression test.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ViktorHofer
ViktorHofer requested a review from JanProvaznik July 1, 2026 06:52
@ViktorHofer
ViktorHofer requested a review from a team as a code owner July 1, 2026 08:13
@ViktorHofer
ViktorHofer enabled auto-merge (squash) July 1, 2026 08:15
@ViktorHofer
ViktorHofer merged commit a81b435 into vs18.9 Jul 1, 2026
12 checks passed
@ViktorHofer
ViktorHofer deleted the backport/pr-14223-to-vs18.9 branch July 1, 2026 09:11
@ViktorHofer ViktorHofer mentioned this pull request Aug 11, 2026
70 tasks
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