copy-tarballs: drop perl bindings

[Mic92]

This hopefully makes it easier to re-write this script in a language that people understand. Because it's shelling out, it's likely slower but hopefully still fast enough for our purposes.

Things done

I only tested local dry run without any credentials...

• 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/)
• 25.05 Release Notes (or backporting 24.11 and 25.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.

[Mic92]

@vcunat can you give me some code how to test it?

[Mic92]

Ok. Looks like I also should replace hashFile

[vcunat]

Nit: is it worth using --argstr when you can directly anti-quote the values into the nix expression? (at least I do assume that it's at least as easy in Perl as in Bash)

[Mic92]

Nit: is it worth using --argstr when you can directly anti-quote the values into the nix expression? (at least I do assume that it's at least as easy in Perl as in Bash)

It's less mental overhead for me to have to think about potential escaping issues, so I find it easier to read this way.

[mweinelt]

Tested and runs into the following error on 4792576

error:
       … while calling the 'map' builtin
         at /home/tarball-mirror/nixpkgs/maintainers/scripts/find-tarballs.nix:26:17:
           25|
           26|   uniqueFiles = map (x: x.file) (genericClosure {
             |                 ^
           27|     startSet = map (file: {
       … while calling the 'genericClosure' builtin
         at /home/tarball-mirror/nixpkgs/maintainers/scripts/find-tarballs.nix:26:34:
           25|
           26|   uniqueFiles = map (x: x.file) (genericClosure {
             |                                  ^
           27|     startSet = map (file: {
       (stack trace truncated; use '--show-trace' to show the full, detailed trace)
       error: attribute 'outputHashAlgo' missing
       at /home/tarball-mirror/nixpkgs/maintainers/scripts/find-tarballs.nix:38:12:
           37|     isPatch = (drv ? postFetch && drv.postFetch != "");
           38|     type = drv.outputHashAlgo;
             |            ^
           39|     name = drv.name;
./maintainers/scripts/copy-tarballs.pl: evaluation failed

[mweinelt]

Bisected and fixed in #414869.

Now I applied this patch for nixos/infra to pluto.nixos.org and mirror-tarballs.service works again.

[mweinelt]

Can you rebase?

[Mic92]

@mweinelt Done