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

Revert MSBuildDebugEngine #6787

Closed

Conversation

rainersigwald
Copy link
Member

This reverts commit cdb5077.

Conflicts:
documentation/wiki/ChangeWaves.md
src/Build/BackEnd/Components/Scheduler/Scheduler.cs
src/Shared/Traits.cs

Fixes a few issues that have cropped up, see #6639 (comment) and some internal Teams conversations.

I have no objection to bringing back the bulk of the change after we figure out the problems--and I believe they're all related to changing the stack-dump directory.

…ngine; Inject binary logger from BuildManager; print static graph as .dot file (dotnet#6639)"

This reverts commit cdb5077.

Conflicts:
	documentation/wiki/ChangeWaves.md
	src/Build/BackEnd/Components/Scheduler/Scheduler.cs
	src/Shared/Traits.cs
@Forgind Forgind added the merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now. label Aug 29, 2021
@rainersigwald
Copy link
Member Author

rainersigwald commented Aug 31, 2021

Closing this because we decided to try fixing forward in #6792.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-when-branch-open PRs that are approved, except that there is a problem that means we are not merging stuff right now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants