Conversation
|
One new test lacks an rpath/ld_preload to run lcoally: UPDATE: filed upstream as https://sourceware.org/PR32929 |
|
Added |
r-burns
left a comment
There was a problem hiding this comment.
LGTM and builds/runs on x86_64-linux
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/5509 |
See NixOS/nixpkgs#402441 Test Plan ========= CI
See NixOS/nixpkgs#402441 Test Plan ========= CI
|
@trofi probably full log: |
|
Good catch! I keep forgetting to test it. Given that patch is a bit invasive let's fine out why |
|
Looks the same. Both start from Ahh! It's because |
|
Upstream fix: https://sourceware.org/git/?p=elfutils.git;a=commitdiff;h=b426c4db31e7c80d4262abdd845d2ece0c9a841c
|
|
This elfutils is most broken library that I ever seen, how do they live with it? |
|
Found a reasonable workaround: build out of source tree to avoid preConfigure = ''
mkdir build-tree
cd build-tree
'';
configureScript = "../configure";Proposed the fix as: |
|
Filed upstream bug report as https://sourceware.org/PR33103 |
Changes: https://www.mail-archive.com/elfutils-devel@sourceware.org/msg08098.html
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.