Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jun 25, 2021

Backport of #54728 to release/6.0-preview6

/cc @hoyosjs

Customer Impact

Reverting as this change breaks several important scenarios in debugging in Linux and macOS.

Testing

Back to heavily tested version - nothing depends on this change other than the bug it tried to fix.

Risk

Very low.

This reverts commit b2fe667.
@ghost
Copy link

ghost commented Jun 25, 2021

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #54728 to release/6.0-preview6

/cc @hoyosjs

Customer Impact

Testing

Risk

Author: github-actions[bot]
Assignees: -
Labels:

area-Diagnostics-coreclr

Milestone: -

@hoyosjs hoyosjs requested review from josalem, sdmaclea and tommcdon June 25, 2021 17:27
@hoyosjs hoyosjs added the Servicing-consider Issue for next servicing release review label Jun 25, 2021
Copy link
Member

@tommcdon tommcdon left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@hoyosjs
Copy link
Member

hoyosjs commented Jun 25, 2021

/azp run dotnet-linker-tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

Approved. We should consider this for Preview 6

@hoyosjs
Copy link
Member

hoyosjs commented Jun 25, 2021

/azp run runtime-dev-innerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jeffschwMSFT
Copy link
Member

@hoyosjs can you double check the failures in CI and then we can merge.

@hoyosjs
Copy link
Member

hoyosjs commented Jun 25, 2021

I've retriggered the leg. The issue is a known in bug in XUnit that's starting to cause us more pronounced pain.

@jeffschwMSFT jeffschwMSFT merged commit 4429fef into release/6.0-preview6 Jun 25, 2021
@hoyosjs hoyosjs deleted the backport/pr-54728-to-release/6.0-preview6 branch June 26, 2021 00:22
@ghost ghost locked as resolved and limited conversation to collaborators Jul 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Diagnostics-coreclr Servicing-consider Issue for next servicing release review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants