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

Add Pybind11 support for orbetto #1

Closed
wants to merge 34 commits into from
Closed

Conversation

lvb2000
Copy link
Contributor

@lvb2000 lvb2000 commented Nov 7, 2023

  • added pybind11 library (git clone)
  • added main wrapper for orbetto called orbethon (orboculum + perfetto + python)
  • added python argparse wrapper for options
  • added meson and ninja build to GitHub Actions

Lukas added 7 commits November 20, 2023 14:25
Note: The elf file could not be parsed in python as the symbol table did not have length entries for strings. That is why the whole .text section is wrapped as python list(uint8_t) and handed over to c++ main function.
@niklaut niklaut marked this pull request as ready for review December 13, 2023 14:37
Copy link
Collaborator

@niklaut niklaut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! Some minor polish, then squash and rebase on top of main.

.github/workflows/build-linux.yml Show resolved Hide resolved
ext/orbetto/meson.build Outdated Show resolved Hide resolved
ext/orbetto/orbethon.cpp Outdated Show resolved Hide resolved
ext/orbetto/orbethon.py Outdated Show resolved Hide resolved
ext/orbetto/orbethon.py Outdated Show resolved Hide resolved
ext/orbetto/orbetto.cpp Show resolved Hide resolved
ext/orbetto/orbetto.cpp Outdated Show resolved Hide resolved
ext/orbetto/orbetto.cpp Outdated Show resolved Hide resolved
ext/orbetto/orbetto.cpp Outdated Show resolved Hide resolved
ext/orbetto/orbetto.cpp Outdated Show resolved Hide resolved
@lvb2000 lvb2000 closed this Jan 10, 2024
@lvb2000 lvb2000 deleted the pybind11 branch January 10, 2024 09:40
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