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

Allow Debugger attributes to be trimmed in Mono #39479

Merged
merged 5 commits into from
Jul 20, 2020

Conversation

eerhardt
Copy link
Member

  • Changed debugger-agent to allow for the Debugger Attributes to not exist.
  • Removed the Debugger Attributes entries from the Descriptors.xml file.

Fix dotnet/linker#1355

Changed debugger-agent to allow for the Debugger Attributes to not exist.
Removed the Debugger Attributes entries from the Descriptors.xml file.

Fix dotnet/linker#1355
@ghost
Copy link

ghost commented Jul 16, 2020

Tagging subscribers to this area: @tommcdon
Notify danmosemsft if you want to be subscribed.

@ghost
Copy link

ghost commented Jul 16, 2020

Tagging subscribers to this area: @thaystg
Notify danmosemsft if you want to be subscribed.

Copy link
Member

@lewing lewing left a comment

Choose a reason for hiding this comment

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

Looks good to me

@lewing
Copy link
Member

lewing commented Jul 17, 2020

Browser failure looks like #39473

@eerhardt
Copy link
Member Author

The linker leg timed out because the OSX build took double the time it normally does. Flakey infrastructure issue.

@eerhardt eerhardt merged commit 0a1f7ba into dotnet:master Jul 20, 2020
@eerhardt eerhardt deleted the RemoveDebuggerAttributes branch July 20, 2020 20:42
Jacksondr5 pushed a commit to Jacksondr5/runtime that referenced this pull request Aug 10, 2020
* Allow Debugger attributes to be trimmed in Mono.

Changed debugger-agent to allow for the Debugger Attributes to not exist.
Removed the Debugger Attributes entries from the Descriptors.xml file.

Fix dotnet/linker#1355

* Use GENERATE_TRY_GET_CLASS_WITH_CACHE to initialize Debugger attribute pointers.
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debugger attributes on methods still left after trimming
5 participants