Skip to content
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

tests.cuda fails to evaluate: "Function called without required argument "backendStdenv" #280377

Closed
trofi opened this issue Jan 11, 2024 · 2 comments

Comments

@trofi
Copy link
Contributor

trofi commented Jan 11, 2024

Describe the bug

Noticed the test evaluation failure when explored nixpkgs attributes for obvious instantiation mistakes.

The reproducer:

$ nix-instantiate '<nixpkgs>' -A tests.cuda
error:
       … while calling the 'abort' builtin

         at lib/customisation.nix:210:13:

          209|        # which is especially relevant with allowAliases = false
          210|        else abort "lib.customisation.callPackageWith: ${error}";
             |             ^
          211|

       error: evaluation aborted with the following error message: 'lib.customisation.callPackageWith: Function called without required argument "backendStdenv" at pkgs/test/cuda/cuda-library-samples/generic.nix:3'

Expected behavior

The test set should instantiate successfully.

Notify maintainers

@ConnorBaker @samuela @SomeoneSerge

Metadata

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.10, NixOS, 24.05 (Uakari), 24.05.20240111.be34eb4`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.19.2`
 - channels(root): `""`
 - channels(slyfox): `""`
 - nixpkgs: `/run/current-system/sw/share/nixos-flakes/inputs/nixpkgs`

Add a 👍 reaction to issues you find important.

@SomeoneSerge
Copy link
Contributor

SomeoneSerge commented Jan 11, 2024

Oh, again, CC #279684 #279703 @NixOS/cuda-maintainers

The immediate error is that this should be cudaPackages_XX.callPackage, not pkgs.callPackage. That said though, the samples are already part of the cudaPackages scope, e.g. cudaPackages_11_8.cuda-library-samples. As mentioned in the referenced issue, we should move them out of pkgs/tests bc right now cudaPackages weirdly depend on pkgs/tests and they shouldn't

@SomeoneSerge
Copy link
Contributor

#280386

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants