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
Merge pull request #75946 from davidtwco/computecpp-init #75946
Conversation
]; | ||
libaries = stdenv.lib.makeLibraryPath [ | ||
stdenv.cc.cc.lib | ||
ncurses5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd appreciate advice from reviewers on how best to fix a issue with this package. When running the binary, I get the following output:
/nix/store/anf6w9ad9577d121w8v2jxhp9m09ycil-computecpp-1.2.0/bin/compute: /nix/store/7fnhs9lg64fgzx13d2skjrckiic2yqsk-ncurses-6.1-20190112-abi5-compat/lib/libtinfo.so.5: no version information available (required by /nix/store/anf6w9ad9577d121w8v2jxhp9m09ycil-computecpp-1.2.0/bin/compute)
I believe this is due to the original binary being compiled with an older version of libtinfo.so.5
, but I'm not sure what I can do about that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My guess is that libtinfo.so
has been stripped of debug symbols. Does this prevent the program from starting?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't, it appears to work fine but just prints that message when invoked.
]; | ||
libaries = stdenv.lib.makeLibraryPath [ | ||
stdenv.cc.cc.lib | ||
ncurses5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My guess is that libtinfo.so
has been stripped of debug symbols. Does this prevent the program from starting?
d760722
to
2354b8d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The derivation looks good to me now but I am no expert in compiler integration. Maybe @bhipple @Ericson2314 @Mic92 @FRidh want to have a look?
Right now it does not find the gcc/stdlibc++ headers:
|
I'll look into fixing this, thanks. |
Signed-off-by: David Wood <david.wood@codeplay.com>
2354b8d
to
853372b
Compare
@Mic92 I've updated the commit to fix that issue. |
Motivation for this change
Adds package for ComputeCpp.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc me!