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

cctk in nixpkgs? #1

Closed
hacker1024 opened this issue Nov 9, 2022 · 6 comments · Fixed by NixOS/nixpkgs#260715
Closed

cctk in nixpkgs? #1

hacker1024 opened this issue Nov 9, 2022 · 6 comments · Fixed by NixOS/nixpkgs#260715

Comments

@hacker1024
Copy link

Hi, thanks for publishing your work!

The cctk package will be very useful, but is a little hard to use on non-flake systems.
Would you consider making a PR to add it into nixpkgs?

@RyanGibb
Copy link
Owner

RyanGibb commented Nov 9, 2022

No problem!

I do want to upstream it, but unfortunately it depends on a fork of patchelf that adds the --replace-symbol command. The author of that fork does plan to upstream it at some point, but there's no ETA. See here for a discussion: https://discourse.nixos.org/t/writing-derivation-for-binary-driver-that-accesses-hard-coded-paths-in-the-fhs/22969/4?u=ryang

@RyanGibb
Copy link
Owner

RyanGibb commented Nov 9, 2022

Looks like the issue is being tracked here NixOS/patchelf#416.

I've tried using LD_PRELOAD and Nixpkgs' libredirect but it doesn't propagate (from cctk) to loading of shared libraries (that cctk uses): https://discourse.nixos.org/t/writing-derivation-for-binary-driver-that-accesses-hard-coded-paths-in-the-fhs/22969/7?u=ryang

@RyanGibb
Copy link
Owner

RyanGibb commented Mar 3, 2023

62a7d09 goes some way towards using upstreamed support for rewriting dynamic symbols, but it's currently failing with:

patchelf: error: Span access out of range.

@RyanGibb
Copy link
Owner

RyanGibb commented Mar 7, 2023

Currently pending on NixOS/patchelf#473, then I can upstream this.

@RyanGibb
Copy link
Owner

RyanGibb commented Nov 7, 2023

Closed by NixOS/nixpkgs#260715

@RyanGibb RyanGibb closed this as completed Nov 7, 2023
@RyanGibb
Copy link
Owner

RyanGibb commented Nov 7, 2023

Didn't even take a year :-)

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 a pull request may close this issue.

2 participants