Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jul 24, 2022

Backport of #72735 to release/7.0-preview7

/cc @hoyosjs

Customer Impact

All debugging on ARM64 Linux and macOS on coreclr will fail at startup without this change.

Testing

Manual testing.

Risk

Very low - reverts to prior behavior.

@ghost
Copy link

ghost commented Jul 24, 2022

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@hoyosjs hoyosjs self-assigned this Jul 24, 2022
@hoyosjs hoyosjs requested a review from a team July 24, 2022 11:17
@ghost
Copy link

ghost commented Jul 24, 2022

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

Issue Details

Backport of #72735 to release/7.0-preview7

/cc @hoyosjs

Customer Impact

All debugging on ARM64 Linux and macOS on coreclr will fail at startup without this change.

Testing

Manual testing.

Risk

Very low - reverts to prior behavior.

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

area-Diagnostics-coreclr

Milestone: -

@hoyosjs hoyosjs added the Servicing-consider Issue for next servicing release review label Jul 24, 2022
@hoyosjs hoyosjs added this to the 7.0.0 milestone Jul 24, 2022
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 will take for consideration for 7.0 preview 7

@jeffschwMSFT
Copy link
Member

@hoyosjs and @tommcdon can you take a look at the failing CI?

@hoyosjs
Copy link
Member

hoyosjs commented Jul 25, 2022

It's WASM linkage - I'll take a look but this change only touches CoreCLR's mscordbi, so it can't be the cause.

@hoyosjs
Copy link
Member

hoyosjs commented Jul 25, 2022

@lewing @steveisok Looks like NPM audits are failing

@hoyosjs hoyosjs merged commit 09b963f into release/7.0-preview7 Jul 25, 2022
@hoyosjs hoyosjs added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jul 25, 2022
@hoyosjs
Copy link
Member

hoyosjs commented Jul 25, 2022

Approved offline.

@hoyosjs hoyosjs deleted the backport/pr-72735-to-release/7.0-preview7 branch July 25, 2022 18:35
@ghost ghost locked as resolved and limited conversation to collaborators Aug 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants