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

suggest add -W para for readelf.py #405

Open
sfncat opened this issue May 19, 2022 · 3 comments
Open

suggest add -W para for readelf.py #405

sfncat opened this issue May 19, 2022 · 3 comments

Comments

@sfncat
Copy link

sfncat commented May 19, 2022

readelf.py -s have the fix length 25
self._emitline('%6d: %s %s %-7s %-6s %-7s %4s %.25s%s' % (
suggest like readelf add -W para, so the ndx string can display all.
-W
--wide
Don't break output lines to fit into 80 columns. By default
readelf breaks section header and segment listing lines for
64-bit ELF files, so that they fit into 80 columns. This
option causes readelf to print each section header resp. each
segment one a single line, which is far more readable on
terminals wider than 80 columns.

@sevaa
Copy link
Contributor

sevaa commented May 19, 2022

As far as I understand, readelf.py only exists so that one may match the parsing results of pyelftools with those of regular readelf. It was never meant as a first class ELF file visualizer.

@WillDenissen
Copy link
Contributor

I have a branch ready that add -W -w support to readelf.py which passes all tests. I'm new however to github, so how do I create a pull request addressing this issue?

@sevaa
Copy link
Contributor

sevaa commented Apr 12, 2024

@sfncat I though we agreed we are not adding -W support to readelf. Let's close this.

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

No branches or pull requests

4 participants