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

Stripping debug info #35

Closed
nmilosev opened this issue Feb 16, 2017 · 5 comments
Closed

Stripping debug info #35

nmilosev opened this issue Feb 16, 2017 · 5 comments

Comments

@nmilosev
Copy link
Contributor

Hello, while packaging for Fedora, I noticed you are stripping debug info in the Makefile.

Could you provide insight why is that so, and maybe consider leaving the debug information there?

Kind regards,
Nemanja

@mptre
Copy link
Owner

mptre commented Feb 16, 2017

I tend to follow the common practices from OpenBSD which is the OS I'm
using. On OpenBSD, third-party software is stripped on installation.

You could, however, opt-out by overwriting the INSTALL_PROGRAM
variable and remove the -s option:

make INSTALL_PROGRAM="install -m 0755" install

@nmilosev
Copy link
Contributor Author

nmilosev commented Feb 16, 2017

Thanks for the clarification. :)

I wrote a small patch which does the same (it was easier for me): https://pagure.io/yank-rpm/raw/makebuild/f/yank-debuginfo.patch

This patch is used when RPM's are built, because in Fedora packages have debuginfo.

Closing this! Thanks again. :)

@mptre
Copy link
Owner

mptre commented Feb 16, 2017

Great! Let me know if anything else can be improved to make packaging
yank more convenient.

@nmilosev
Copy link
Contributor Author

Maybe add a BINARY_NAME variable in the Makefile, so we can override in case we need to work around packages with /usr/bin/yank.

Relevant patch for Fedora: https://pagure.io/yank-rpm/raw/makebuild/f/yank-name-change.patch

I can try and send you a PR if you want. :)

@mptre
Copy link
Owner

mptre commented Feb 16, 2017

I can try and send you a PR if you want. :)

Please do. Call the variable PROG and limit its usage to the name of
binary and not the man-page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants