nv-codec-headers: recreate under by-name

[AndersonTorres]

Description of changes

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[AndersonTorres]

I will split handbrake in its own PR: #324418

[Aleksanaa]

Could you split migration and format in two commits to have a good log?

[AndersonTorres]

Formatting? This is a whole new derivation written basically from scratch that does the same as the previous one.
There is no nixfmt happening here.

Any "intermediate steps" would be broken by definition.

[Aleksanaa]

Then I have to say that your commit message is a bit misleading 😅

[AndersonTorres]

I will reformulate as soon as possible.

[AndersonTorres]

@Aleksanaa done.

[flokli]

It looks like this broke eval:

error:
       … while calling the 'head' builtin

         at /nix/store/pps0ryglnbx1p277ghzv4ax1is1lsb3c-nixpkgs-src/lib/attrsets.nix:1575:11:

         1574|         || pred here (elemAt values 1) (head values) then
         1575|           head values
             |           ^
         1576|         else

       … while evaluating the attribute 'value'

         at /nix/store/pps0ryglnbx1p277ghzv4ax1is1lsb3c-nixpkgs-src/lib/modules.nix:821:9:

          820|     in warnDeprecation opt //
          821|       { value = addErrorContext "while evaluating the option `${showOption loc}':" value;
             |         ^
          822|         inherit (res.defsFinal') highestPrio;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'nv-codec-headers-12' missing

       at /nix/store/pps0ryglnbx1p277ghzv4ax1is1lsb3c-nixpkgs-src/pkgs/by-name/nv/nv-codec-headers/package.nix:16:12:

           15|     "11" = sources.nv-codec-headers-11;
           16|     "12" = sources.nv-codec-headers-12;
             |            ^
           17|   }.${majorVersion};

[flokli]

Revert in #325242.

[Aleksanaa]

I don't quite get it:

=> nix build .#nv-codec-headers-12
path '/home/aleksana/works/nixpkgs/pkgs/by-name/nv/nv-codec-headers' does not contain a 'flake.nix', searching up

=> tree ./result/
./result/
├── include
│   └── ffnvcodec
│       ├── dynlink_cuda.h
│       ├── dynlink_cuviddec.h
│       ├── dynlink_loader.h
│       ├── dynlink_nvcuvid.h
│       └── nvEncodeAPI.h
└── lib
    └── pkgconfig
        └── ffnvcodec.pc

5 directories, 6 files

Looks normal?

[Aleksanaa]

This PR only drops nv-codec-headers-versions, but that's not used anywhere within Nixpkgs either.

[AndersonTorres]

Oh boi, just when I was updating nvch 12...