-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
nv-codec-headers: adopt and refactor #287918
nv-codec-headers: adopt and refactor #287918
Conversation
3e095a8
to
090af84
Compare
ffb5b37
to
909ecfb
Compare
@infinisil it looks like a bug to me! Can you verify? I am using an external file to parameterize the |
Not a bug I'm afraid! To CI, the Since inherit (callPackages ../development/libraries/nv-codec-headers { })
nv-codec-headers-9 nv-codec-headers-10; or if you want to also expose the set of versions: nv-codec-headers-versions = callPackages ../development/libraries/nv-codec-headers { };
inherit (nv-codec-headers-versions)
nv-codec-headers-9 nv-codec-headers-10; See also #285922 (comment) or #287081 for some discussion on this. |
909ecfb
to
3830838
Compare
|
3830838
to
0371345
Compare
0371345
to
4f571b4
Compare
1b8f8fa
to
eb341d3
Compare
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/3486 |
eb341d3
to
caeba37
Compare
It globs all versions of nv-codec-headers kept/needed on Nixpkgs. The default is kept at 9, as before.
caeba37
to
719105a
Compare
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-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.