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

Update Dyninst submodule #303

Open
jrmadsen opened this issue Aug 30, 2023 · 0 comments
Open

Update Dyninst submodule #303

jrmadsen opened this issue Aug 30, 2023 · 0 comments
Labels
Dyninst Involves Dyninst

Comments

@jrmadsen
Copy link
Collaborator

Due to the delay in the acceptance of the cmake modifications to Dyninst (~1.5 years), omnitrace eventually ended up with a fork of Dyninst that became extremely difficult to sync. Even moreso because of the ill-fated decision to apply source formatting. Furthermore, when Dyninst did update their cmake, they removed the option is install the dependencies. Thus the following steps need to be taken to bring Dyninst up to date:

  • Move the options to build TBB, Boost, ElfUtils, and LibIberty into Omnitrace as this is important for packaging an installation of omnitrace that does not require the user to separately install these libs
  • Update the Dyninst submodule to clone their releases
  • Change all references to DYNINST_BUILD_<DEP> to be OMNITRACE_BUILD_<DEP> in the various scripts
  • Patch the base docker images used for CI (which pre-install Dyninst) in the absence of there being a Dyninst cmake option to build the deps
@jrmadsen jrmadsen added the Dyninst Involves Dyninst label Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dyninst Involves Dyninst
Projects
None yet
Development

No branches or pull requests

1 participant