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

--show-needed option #67

Closed
wants to merge 2 commits into from

Conversation

AaronDMarasco-VSI
Copy link
Contributor

To complement the add/remove/replace options of DT_NEEDED entries.

Otherwise I was doing things with readelf and grep.

I am not an ELF expert in any way, so if this was already possible with some of the other options, please advise.

Aaron D. Marasco added 2 commits September 14, 2015 07:36
Hacked removeNeeded() interface a little to get it cheaply.
@darealshinji
Copy link
Contributor

I think this can be closed now that --print-needed was implemented and is part of release 0.9.

@AaronDMarasco-VSI
Copy link
Contributor Author

Works for me. There was also a small memory leak fix 28b99d1 that might still be wanted (I haven't checked lately).

@domenkozar
Copy link
Member

Closing as fixed. Please submit a PR for memory leak fix if still applicable.

@AaronDMarasco
Copy link

Confirmed the memory leak was fixed elsewhere sometime in the past 4+ years; looks like a lot of memory management moved to Modern C++.

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

Successfully merging this pull request may close these issues.

None yet

4 participants