nixpkgs-update-fetch-updatescript is broken

[zowoq]

cc @ryantm @rhendric

This fetcher seems to depend on being able to evaluate all of the updateScript's in nixpkgs without error. I've fixed a broken updateScript in nixpkgs once already for this fetcher but it'll probably be better if we could change it to ignore errors. Making ofborg enforce evaluation of updateScript might also be a possibility if ignoring errors isn't feasible?

Current error:

error: Package ‘cuda_cudart-11.8.89’ in nixpkgs/pkgs/development/cuda-modules/generic-builders/manifest.nix:325 has an unfree license (‘CUDA EULA’), refusing to evaluate.

With export NIXPKGS_ALLOW_UNFREE=1:

error: paddlepaddle has no binary-hashes.nix entry for 'x86_64-linux.cpu.cp311' attribute

[zowoq]

Seems to have been fixed in nixpkgs but I'll leave this issue open until we make the fetcher able to handle nixpkgs breakage or we decide making against any changes here and just keep fixing breakage in nixpkgs as it is occurs.

[zowoq]

Broken again:

error:
       … while evaluating the file '/nix/store/ay6fkwf1ivcgkz2zd0sm3w300gq6zwm8-packages-with-update-script.nix':
       … while calling the 'concatStringsSep' builtin
         at /nix/store/hjn3x00py1v9v28f9rfvd3yp4v1s5m01-source/lib/strings.nix:60:31:
           59|   */
           60|   concatMapStrings = f: list: concatStrings (map f list);
             |                               ^
           61|
       … while calling the 'map' builtin
         at /nix/store/hjn3x00py1v9v28f9rfvd3yp4v1s5m01-source/lib/strings.nix:60:46:
           59|   */
           60|   concatMapStrings = f: list: concatStrings (map f list);
             |                                              ^
           61|
       (stack trace truncated; use '--show-trace' to show the full trace)
       error: function 'anonymous lambda' called with unexpected argument 'rev'
       at /nix/store/hjn3x00py1v9v28f9rfvd3yp4v1s5m01-source/pkgs/common-updater/git-updater.nix:6:1:
            5|
            6| { pname ? null
             | ^
            7| , version ? null

[zowoq]

error:
       … while evaluating the file '/nix/store/ay6fkwf1ivcgkz2zd0sm3w300gq6zwm8-packages-with-update-script.nix':
       … while calling the 'concatStringsSep' builtin
         at /nix/store/sd1cc3syrldcjws0k3z5qszmpi2n0j53-source/lib/strings.nix:95:31:
           94|   */
           95|   concatMapStrings = f: list: concatStrings (map f list);
             |                               ^
           96|
       … while calling the 'map' builtin
         at /nix/store/sd1cc3syrldcjws0k3z5qszmpi2n0j53-source/lib/strings.nix:95:46:
           94|   */
           95|   concatMapStrings = f: list: concatStrings (map f list);
             |                                              ^
           96|
       (stack trace truncated; use '--show-trace' to show the full trace)
       error: function 'anonymous lambda' called with unexpected argument 'tagPrefix'
       at /nix/store/sd1cc3syrldcjws0k3z5qszmpi2n0j53-source/pkgs/common-updater/git-updater.nix:6:1:
            5|
            6| { pname ? null
             | ^
            7| , version ? null

[zowoq]

@infinisil I thought I'd ask you about this as you are working on the github actions eval checks.

Seems to be every few weeks an updateScript that doesn't evaluate is merged in nixpkgs, we then need to bisect and fix it as it breaks the service that generates a list of all the updateScripts for the nixpkgs-update bot.

We could try handling the eval failures on our side (in this file) and maybe even publish a list of packages with a broken updateScript but I expect we'd still get questions along the lines of "why isn't the bot working for my package".

How feasible would it be to extend the nixpkgs eval checks to enforce evaluation of passthru.updateScript?