-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add ability to retain debuginfo before stripping binaries #49
base: main
Are you sure you want to change the base?
Conversation
Thanks for the PR! How do you plan to distribute the .debug file? Currently it is not included in the archive. (Here is code to include binary to archive.) Also, is there any reason to prefer the objcopy command over the cargo profile option? I don't think that would work well with cross-compilation. (like #8) |
Right, I was thinking people could use the
Great question! The issue with the cargo profile option is that it doesn't produce the same data. It produces DWARF packages, which are nowhere near as well supported by tooling as regular DWARF. I've used |
Any progress on this PR? Being able to upload a separate debug info archive would be great. |
If you use the cargo profile mentioned in #59 and one of the The main issue with this PR is that it is implemented in a way other than the cargo profile. We know from experience that that fact causes issue. Also it may not work correctly if strip option in cargo profile is specified. And in that case, it means that this will no longer work by default on 1.77+ (rust-lang/cargo#13257). |
Resolves #48