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

ImageTool: Add debug symbols path argument #52

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

savvamitrofanov
Copy link
Contributor

We need to have the ability to put custom path with binary containing debug symbols into ImageTool, because currently it uses the path to stripped binary without debug symbols and that's why debugging is broken for targets built with GCC, CLANGDWARF toolchains.

We need to have the ability to put custom path with binary containing
debug symbols into ImageTool, because currently it uses the path to
stripped binary without debug symbols

Signed-off-by: Savva Mitrofanov <[email protected]>
@savvamitrofanov savvamitrofanov merged commit 87db423 into master Aug 30, 2023
53 checks passed
@savvamitrofanov savvamitrofanov deleted the savvas-imagetool-symbolspath-20230830 branch November 16, 2023 20:42
MikhailKrichanov pushed a commit that referenced this pull request Dec 15, 2023
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (#69)
54e5bd1 Enable GTK on Fedora QEMU (#63)
f1c7a20 Fedora: install code coverage tools for GCC (#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (#61)
14d2aba Add Fedora 37 image with gcc12 (#60)
5b8a008 Add dotnet runtime to fedora build (#57)
f5c874a Fix platform build file name for EDK2 change (#58)
48540ad Ubuntu-20: Fix dev image entrypoint (#55)
98e849d Fedora-35: Add Powershell to build image (#52)

Cc: Michael D Kinney <[email protected]>
Cc: Sean Brogan <[email protected]>
Cc: Michael Kubacki <[email protected]>
Cc: Oliver Steffen <[email protected]>
Cc: Chris Fernald <[email protected]>
Signed-off-by: Gua Guo <[email protected]>
Reviewed-by: Michael D Kinney <[email protected]>
Reviewed-by: Michael Kubacki <[email protected]>
Reviewed-by: Chris Fernald <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant