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

Respect EnableUnmanagedDebugging property to enable native debugging #6176

Open
ViktorHofer opened this issue May 8, 2020 · 0 comments
Open
Labels
Parity-Legacy-Feature Missing features from the legacy project system. Triage-Investigate Reviewed and investigation needed by dev team
Milestone

Comments

@ViktorHofer
Copy link
Member

Visual Studio Version: 16.7 P1

Summary:
The EnableUnmanagedDebugging property isn't honored for .NET Core projects and launchSettings.json files need to be created just for the sake of enable native debugging: #1125 (comment).

Steps to Reproduce:

  1. Create a .NET Core project which launches a sub process in which you want to set a breakpoint.

  2. Add the EnableUnmanagedDebugging property and set it to true.

  3. F5 the project and notice that the module isn't loaded and the breakpoint isn't hit.

Expected Behavior:
Setting the EnableUnmanagedDebugging property should be honored by the project system.

<Project>
  <PropertyGroup>
    <TargetFramework>net5.0</TargetFramework>
    <EnableUnmanagedDebugging>true</EnableUnmanagedDebugging>
  </PropertyGroup>
  ...
</Project>

Actual Behavior:
The property isn't honored and a launchSettings.json needs to be created (either manually or by checking the "Enable native code debugging" checkbox in the Debug pane).

User Impact:
launchSettings.json files need to be checked in for every project to honor that setting or need to be created on the fly. For projects like dontet/runtime the former is not doable because of the huge amount of projects and the latter is cumbersome as the launchSettings file needs to be placed in the Properties folder which is a candidate for git clean -xdf.

cc @davkean

@jjmew jjmew added Triage-Investigate Reviewed and investigation needed by dev team Parity-Legacy-Feature Missing features from the legacy project system. labels May 12, 2020
@jjmew jjmew added this to the Backlog milestone May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Parity-Legacy-Feature Missing features from the legacy project system. Triage-Investigate Reviewed and investigation needed by dev team
Projects
None yet
Development

No branches or pull requests

2 participants