Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Revert "Debugger api to set a breakpoint on offset 0 of all methods" #16300

Merged
merged 2 commits into from
Feb 9, 2018

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented Feb 9, 2018

This introduced OSX build break.

@jkotas
Copy link
Member Author

jkotas commented Feb 9, 2018

@davmason I am reverting your change because of it broke the build on OSX. The OSX leg on your PR was clearly failing:

image

Please do not merge without attributing CI failures to know https://github.com/dotnet/coreclr/issues?q=is%3Aissue+is%3Aopen+label%3Ablocking-clean-ci issues.

cc @noahfalk @lt72

@jkotas jkotas merged commit 8d4d913 into dotnet:master Feb 9, 2018
@jkotas jkotas deleted the revert branch February 10, 2018 01:01
@sandreenko
Copy link

How did you decide that this PR caused OSX problems? As I see from pr triggered OSX several jobs before the original PR was merged also had failed.

Please return this fix back.
cc @dotnet/arm64-contrib

@jkotas
Copy link
Member Author

jkotas commented Feb 10, 2018

The main change was added back with a fix for the build break in #16303 already. However, the follow up fix for the build warning was not.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants