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

Don't create a temporary copy to not break hardlinks (fixes #32) #48

Merged
merged 2 commits into from
Jul 23, 2015

Conversation

darealshinji
Copy link
Contributor

See #32

@edolstra edolstra merged commit 80e25cd into NixOS:master Jul 23, 2015
@darealshinji darealshinji deleted the patch-1 branch July 23, 2015 23:20
@sjackman
Copy link

@darealshinji I've just run into this issue once again. Will you release a new version of patchelf that includes this fix?

@darealshinji
Copy link
Contributor Author

This pull request was merged, so if you use an upstream snapshot that issue shouldn't happen.

@sjackman
Copy link

Yes, but I'd prefer to use a stable release rather than a snapshot. If HEAD is ready to go, could @edolstra please tag a stable release?

@darealshinji
Copy link
Contributor Author

Here's a backport of v0.8: https://github.com/darealshinji/patchelf/tree/backport
You might want to remove src/elf.h to use the system header before compiling.

PS: looking at all the changes since v0.8 it would really be about time to tag a new release.

@sjackman
Copy link

Thanks!

hyperair added a commit to hyperair/kafka-cffi that referenced this pull request Jun 10, 2020
This gets us patchelf 0.10 for free, which fixes
NixOS/patchelf#48 which causes patchelf to struggle with
openssl libraries that are installed with 555 permissions.
hyperair added a commit to hyperair/kafka-cffi that referenced this pull request Jun 15, 2020
This gets us patchelf 0.10 for free, which fixes
NixOS/patchelf#48 which causes patchelf to struggle with
openssl libraries that are installed with 555 permissions.
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.

3 participants