Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[9.0.2xx] Workaround for "MSB4166: Child node "1" exited prematurely for build check #11378

Open
wants to merge 2 commits into
base: vs17.13
Choose a base branch
from

Conversation

YuliiaKovalova
Copy link
Member

@YuliiaKovalova YuliiaKovalova commented Feb 3, 2025

Backporting of #11353

Summary

BuildCheck can still emit some LogBuildEvent(s) after ProjectFinishedEventArgs was reported and entries from _projectFileMap were cleaned up.
Due to GetAndVerifyProjectFileFromContext validation, these checks break the build for the middle/large size projects.
It was discovered on the attempt to dogfood BuildCheck.

Changes Made

If BuildCheck is enabled, _projectFileMap won't be cleaned up.

Customer Impact

Failure of the build in buildcheck is enabled (for middle and large project sizes).

Testing

Local testing with the patched msbuild.

Risk

low: the feature is disabled by default and we need this fix to drive adaptation across dotnet org.

@YuliiaKovalova YuliiaKovalova changed the base branch from main to vs17.13 February 3, 2025 13:43
@YuliiaKovalova YuliiaKovalova requested a review from a team as a code owner February 3, 2025 16:24
@YuliiaKovalova YuliiaKovalova changed the title Workaround for "MSB4166: Child node "1" exited prematurely for build check [9.0.3xx] Workaround for "MSB4166: Child node "1" exited prematurely for build check Feb 4, 2025
@YuliiaKovalova YuliiaKovalova changed the title [9.0.3xx] Workaround for "MSB4166: Child node "1" exited prematurely for build check [9.0.2xx] Workaround for "MSB4166: Child node "1" exited prematurely for build check Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant