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
opencl-info: fix build #98054
opencl-info: fix build #98054
Conversation
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.
there's nothing nixos-specific in this, please upstream the changes so that it's in a better state
Created marchv/opencl-info#2. TBH not sure would it be merged upstream as opencl-info didn't see any updates for 6 years. |
If |
@danieldk |
Where are we with this then? This builds for me (non-nixos linux x86_64) and there are a number of reverse-dependencies stuck on it. |
b2f51bf
to
26d930c
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.
It doesn't seem that anything is going to happen upstream, so let's merge this with the patch. I have added a reference to the PR in the patch comment. If this is unmaintained, we should probably remove it at some point. clinfo
is better anyway and maintained.
Result of nixpkgs-review pr 98054
1
1 package marked as broken and skipped:
- ethminer
2 packages built:
- ethash
- opencl-info
Thanks @demyanrogozhin for fixing this derivation!
When reviewing NixOS#98054, I noticed that this package has a bunch of buildInputs that are unused. There are no references to OpenCL, mesa, boost, cryptopp, or openmpi in the source code. The package compiles fine with these buildInputs removed.
Motivation for this change
The build of opencl-info recently started failing because opencl-clhpp was
updated. The latest version of opencl-hpp does not ship the deprecated
cl.hpp header anymore.
ZHF: #97479
Things done
Tested using sandboxing (nix.useSandbox on NixOS, or option
sandbox
innix.conf
on non-NixOS linux)Built on platform(s)
Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
Tested compilation of all pkgs that depend on this change using
nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
Tested execution of all binary files (usually in
./result/bin/
)Determined the impact on package closure size (by running
nix path-info -S
before and after)Ensured that relevant documentation is up to date
Fits CONTRIBUTING.md.