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

Make symbolizer-rs a library rather than an executable #2

Closed
hugsy opened this issue May 6, 2024 · 5 comments · Fixed by #3
Closed

Make symbolizer-rs a library rather than an executable #2

hugsy opened this issue May 6, 2024 · 5 comments · Fixed by #3
Labels
enhancement New feature or request

Comments

@hugsy
Copy link

hugsy commented May 6, 2024

Improvement suggestion: symbolizer-rs could be beneficial as a library (with the executable parser as an example) would be really convenient for integration to other projects.

@0vercl0k
Copy link
Owner

0vercl0k commented May 6, 2024

Yeah I think that'd be good. I'm going to wait a bit to iron out the first few bugs but when the dust settles, let's talk :)

@0vercl0k 0vercl0k added the enhancement New feature or request label May 6, 2024
@0vercl0k
Copy link
Owner

I think I'll look at that this month hopefully @hugsy - let me know if you have specific requests / usages in mind so that I can do my best to account for it :)

Cheers

@hugsy
Copy link
Author

hugsy commented May 14, 2024

let me know if you have specific requests / usages in mind

Considering it's the only lib (AFAIK) that's cross-platform to symbolize traces, it'd really nice to use it in other languages than Rust, at least with C/C++ via FFI and ideally Python. Personally my (immediate) usage would be Python but I realize it could be too much to ask, so if you can make a lib (first) with FFI (second), I'm happy to contribute and PR some Py bindings.

What do you think ?

@0vercl0k
Copy link
Owner

0vercl0k commented May 15, 2024

Ack yeah, let me first find time make the lib as you mentioned : D

@0vercl0k 0vercl0k linked a pull request May 30, 2024 that will close this issue
@0vercl0k
Copy link
Owner

Okay I've pushed up addr-symbolizer - sorry for the long delay (I lost motivation on the way 😅). There are probably a bunch of rough edges but .. yeah!

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants