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

estrip: update for modern elfutils by passing -R #1096

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

thesamesam
Copy link
Member

Since elfutils 0.170, eu-strip supports -R / --remove-section option [0] """
strip: Add -R, --remove-section=SECTION and --keep-section=SECTION. """

"--remove-comment" is the same as "-R .comment". (eu-strip still does not support -N / --strip-symbol option, which would be good to use when it is implemented (Bug #686282).)

<elfutils-0.170 was deleted almost 2 years ago:
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7a0474559d4361a75a90aadc2b0f5ec46d85f359

Call eu-strip with the same -R options as for GNU strip / llvm-strip:
-R .comment -R .GCC.command.line -R .note.gnu.gold-version

[0] https://sourceware.org/git/?p=elfutils.git;a=blob;f=NEWS;h=3d097c68438489b028e2eb6931f8287fd9a46652;hb=HEAD#l197

Bug: https://bugs.gentoo.org/728100

Since elfutils 0.170, eu-strip supports -R / --remove-section option [0]
"""
strip: Add -R, --remove-section=SECTION and --keep-section=SECTION.
"""

"--remove-comment" is the same as "-R .comment". (eu-strip still does not support
-N / --strip-symbol option, which would be good to use when it is implemented (Bug #686282).)

<elfutils-0.170 was deleted almost 2 years ago:
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7a0474559d4361a75a90aadc2b0f5ec46d85f359

Call eu-strip with the same -R options as for GNU strip / llvm-strip:
  -R .comment -R .GCC.command.line -R .note.gnu.gold-version

[0] https://sourceware.org/git/?p=elfutils.git;a=blob;f=NEWS;h=3d097c68438489b028e2eb6931f8287fd9a46652;hb=HEAD#l197

Bug: https://bugs.gentoo.org/728100
Signed-off-by: Sam James <[email protected]>
@thesamesam
Copy link
Member Author

cc @Arfrever

@thesamesam thesamesam marked this pull request as draft September 17, 2023 07:32
@thesamesam
Copy link
Member Author

Not tested yet.

Signed-off-by: Sam James <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant