Skip to content

nv-codec-headers-12: 12.0.16.0 -> 12.1.14.0#260538

Closed
r-ryantm wants to merge 1 commit intoNixOS:masterfrom
r-ryantm:auto-update/nv-codec-headers-12
Closed

nv-codec-headers-12: 12.0.16.0 -> 12.1.14.0#260538
r-ryantm wants to merge 1 commit intoNixOS:masterfrom
r-ryantm:auto-update/nv-codec-headers-12

Conversation

@r-ryantm
Copy link
Contributor

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/nv-codec-headers/versions.

meta.description for nv-codec-headers-12 is: FFmpeg version of headers for NVENC

meta.homepage for nv-codec-headers-12 is: https://git.videolan.org/?p=ffmpeg/nv-codec-headers.git

Updates performed
  • Version update
To inspect upstream changes
Impact
Checks done (click to expand)
  • built on NixOS
  • The tests defined in passthru.tests, if any, passed
  • found 12.1.14.0 with grep in /nix/store/d4c0bm64gk017dzzvzbp2ngm18ihkk6l-nv-codec-headers-12.1.14.0
  • found 12.1.14.0 in filename of file in /nix/store/d4c0bm64gk017dzzvzbp2ngm18ihkk6l-nv-codec-headers-12.1.14.0

Rebuild report (if merged into master) (click to expand)
3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
jellyfin
jellyfin-ffmpeg
nv-codec-headers-12
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/d4c0bm64gk017dzzvzbp2ngm18ihkk6l-nv-codec-headers-12.1.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A nv-codec-headers-12 https://github.com/r-ryantm/nixpkgs/archive/d4283a97fdd63f650bfcd29d6cf932b91204380b.tar.gz

Or:

nix build github:r-ryantm/nixpkgs/d4283a97fdd63f650bfcd29d6cf932b91204380b#nv-codec-headers-12

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/d4c0bm64gk017dzzvzbp2ngm18ihkk6l-nv-codec-headers-12.1.14.0
ls -la /nix/store/d4c0bm64gk017dzzvzbp2ngm18ihkk6l-nv-codec-headers-12.1.14.0/bin


Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:
  • jellyfin
  • jellyfin-ffmpeg
1 package built:
  • nv-codec-headers-12

Maintainer pings

cc @MP2E for testing.

@ofborg ofborg bot requested a review from MP2E October 11, 2023 22:37
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Oct 11, 2023
@wegank
Copy link
Member

wegank commented Oct 28, 2023

@ofborg build jellyfin

@arthsmn arthsmn mentioned this pull request Nov 26, 2023
13 tasks
@arthsmn
Copy link
Member

arthsmn commented Nov 26, 2023

So, is there anything missing to merge this request? I could only build ffmpeg 6.1 with this update.

@nixos-discourse
Copy link

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/2979

@Atemu
Copy link
Member

Atemu commented Nov 26, 2023

@ofborg build jellyfin-ffmpeg

If you can somehow confirm this is still working, I'll merge.

@Atemu
Copy link
Member

Atemu commented Nov 26, 2023

This appears to break the only dependent package, so this PR won't be merged any time soon.

@arthsmn
Copy link
Member

arthsmn commented Nov 26, 2023

This appears to break the only dependent package, so this PR won't be merged any time soon.

It seems that the jellyfin-ffmpeg is using ffmpeg 6.0, so that is the issue. We need to wait until they update to ffmpeg 6.1 to be able to proceed.

@arthsmn
Copy link
Member

arthsmn commented Nov 27, 2023

Upstream agreed to update to ffmpeg 6.0.1, which has a fix for this.

@Atemu
Copy link
Member

Atemu commented Nov 27, 2023

Thank you very much for bringing this upstream :). It looks like they've already made a release.

Could you create a new PR against staging which updates ffmpeg, ffmpeg-jellyfin and incorporates this PR?

@arthsmn
Copy link
Member

arthsmn commented Dec 5, 2023

Closing as it has been done here: #271905.

@arthsmn arthsmn closed this Dec 5, 2023
@r-ryantm r-ryantm deleted the auto-update/nv-codec-headers-12 branch December 6, 2023 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants