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.

Sign up for GitHub

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

Jump to bottom

Allow multiple modifications in same call #361

Merged
merged 6 commits into from
Dec 22, 2021
Merged

Allow multiple modifications in same call #361

merged 6 commits into from
Dec 22, 2021

Commits on Dec 21, 2021

  1. Allow multiple modifications in same call 

    `patchelf` previously would incorrectly patch the ELF header if it was
called with multiple changes at once such as _add_ & _replace_.

In order to support that, rewrite the sections in between each section
modification.

Fix NixOS#359
    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    d306abe View commit details
    Browse the repository at this point in the history

  2. Add a unit test

    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    e268662 View commit details
    Browse the repository at this point in the history

  3. Remove result and add it to gitignore

    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    dc375bc View commit details
    Browse the repository at this point in the history

  4. Code clarity and test working with musl 

    Made changes according to feedback from @Mic92 such as moving the
`rewriteSections` inline into every method.

Improved `replace-add-needed.sh` to work with musl libc
    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    d02a3ec View commit details
    Browse the repository at this point in the history

  5. Merge branch 'master' into faridzakaria/fix-add-replace-for-real

    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    e589fe3 View commit details
    Browse the repository at this point in the history

  6. Fix replace-add-needed.sh test for musl

    @fzakaria
    fzakaria committed Dec 21, 2021
    Configuration menu
    Copy the full SHA
    b92a6e8 View commit details
    Browse the repository at this point in the history