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

Check soname in dictionary #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Nuzhny007
Copy link

Hi!
I have an errorr with ld-linux-aarch64.so.1:

$python3 copydeps.py ~/work/tm/traffic_monitor/build/traffic_monitor -d ./tm
Traceback (most recent call last):
  File "copydeps.py", line 245, in <module>
    sys.exit(main())
  File "copydeps.py", line 228, in main
    app.run(args.executable)
  File "copydeps.py", line 138, in run
    self._traverse_tree(binary)
  File "copydeps.py", line 165, in _traverse_tree
    self._traverse_tree(soname)
  File "copydeps.py", line 165, in _traverse_tree
    self._traverse_tree(soname)
  File "copydeps.py", line 165, in _traverse_tree
    self._traverse_tree(soname)
  [Previous line repeated 2 more times]
  File "copydeps.py", line 143, in _traverse_tree
    path = self.path_for_binary[binary]
KeyError: 'ld-linux-aarch64.so.1'

And I added some condition then soname exists in self.path_for_binary

@agateau
Copy link

agateau commented Apr 13, 2022

Hi! I think you can achieve in a more generic way this by broadening the DEFAULT_EXCLUDE_LIST from https://github.com/Genymobile/copydeps/blob/master/copydeps.py#L26. Replacing ld-linux-x86-64.so.* with ld-linux-*.so.* should do the job. Can you give it a try?

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.

2 participants