Fix BarViz debugging#5276
Conversation
There was a problem hiding this comment.
Pull Request Overview
Fixes debugging issues with the BarViz project by changing the compiler debug type configuration from "Full" to "portable" to enable proper debugger connection in Visual Studio and VS Code.
- Updates
DebugTypecompiler property from "Full" to "portable" - Adds
DebugSymbolsproperty set to true for complete debug symbol generation - Updates documentation comments with Microsoft reference and clearer explanation
| If set to anything but `portable` VS and VS Code will fail to connect to a running BarViz debug process | ||
| due to maybe some missing debug symbols or attributes. Either way the documentation linked also recommends having | ||
| `portable` instead of `full`, so no harm should be done |
There was a problem hiding this comment.
The comment contains uncertain language ('maybe some missing debug symbols') and informal tone ('Either way', 'so no harm should be done'). Consider rewriting with more definitive technical language, such as: 'Setting to portable ensures compatibility with modern debuggers in VS and VS Code. The portable format is the recommended debug format per Microsoft documentation.'
| If set to anything but `portable` VS and VS Code will fail to connect to a running BarViz debug process | |
| due to maybe some missing debug symbols or attributes. Either way the documentation linked also recommends having | |
| `portable` instead of `full`, so no harm should be done | |
| Setting `DebugType` to `portable` ensures compatibility with modern debuggers in Visual Studio and VS Code. | |
| The Microsoft documentation linked above recommends using the `portable` format instead of `full`. | |
| This setting is required for reliable debugging of the BarViz project. |
| `portable` instead of `full`, so no harm should be done | ||
| --> | ||
| <DebugType>Full</DebugType> | ||
| <DebugType>portable</DebugType> |
There was a problem hiding this comment.
The value should be capitalized as 'Portable' to match standard MSBuild property value conventions, similar to how the previous value was 'Full' rather than 'full'.
| <DebugType>portable</DebugType> | |
| <DebugType>Portable</DebugType> |
Change
DebugTypecompiler prop to fix BarViz not being debuggable in VS and VS Code