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

[C++][Packaging] Package gdb plugin #30904

Closed
asfimport opened this issue Jan 24, 2022 · 7 comments
Closed

[C++][Packaging] Package gdb plugin #30904

asfimport opened this issue Jan 24, 2022 · 7 comments

Comments

@asfimport
Copy link
Collaborator

asfimport commented Jan 24, 2022

The gdb plugin should be packaged appropriately. I'm not sure where/under which form.

For example, on Ubuntu, GDB auto-load scripts are under /usr/share/gdb/auto-load/....
So, if a Arrow package installs /usr/lib/x86_64-linux-gnu/libarrow.so.700.0.0, the gdb_arrow.py script could be installed as /usr/share/gdb/auto-load/usr/lib/x86_64-linux-gnu/libarrow.so.700.0.0-gdb.py.

(take the realpath of the installed DLL, prefix it with the GDB scripts-directory such as /usr/share/gdb/auto-load/, and append -gdb.py at the end of the filename)

Reporter: Antoine Pitrou / @pitrou
Assignee: Kouhei Sutou / @kou

Related issues:

PRs and other links:

Note: This issue was originally created as ARROW-15422. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
cc @kou

@asfimport
Copy link
Collaborator Author

Kouhei Sutou / @kou:
Note: We can implement /usr/share/gdb/auto-load/usr/lib/x86_64-linux-gnu/libarrow.so.700.0.0-gdb.py by import like GLib's -gdb.py does:


import sys
import gdb

dir_ = '/usr/share/arrow/gdb'
if not dir_ in sys.path:
    sys.path.insert(0, dir_)

import gdb_arrow

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Indeed, that is also possible.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Also cc @xhochy . I don't know if conda-forge usually packages gdb extensions?

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
@kou @raulcd One of you may want to tackle this? :-)

@asfimport
Copy link
Collaborator Author

Kouhei Sutou / @kou:
I'll add this to .deb/.rpm packages.

@asfimport
Copy link
Collaborator Author

Kouhei Sutou / @kou:
Issue resolved by pull request 13477
#13477

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

2 participants