I'm interested in how you would now override stuff. Maybe a short example in the commit message?

I'm interested in how you would now override stuff. Maybe a short example in the commit message?

What I've done in a local repo today was:

{
  nixpkgs.overlays = [
    (self: super: { inherit (super.callPackage (super.path + "/pkgs/…/cudatoolkit") {
        addOpenGLRunpath = …;
     }) cudatoolkit_10; })
  ];
}

The probably ugliest part about this is that super.path has to be used and I've to re-import the package definition. Ideally each of the derivations would allow overriding.

Instead of the above super.path hack you can now do:

{
  nixpkgs.overlays = [
    (self: super: { inherit (super.cudatoolkitPackages.override {
        addOpenGLRunpath = …;
     }) cudatoolkit_10; })
  ];
}

Ideally we would be able to override each package. That is something I will work on in the next days once the more pressuring work has been done.

@knedlsepp I did the overridable changes for cudaPackages. Does that look good to you? I am going to do the same for cudnn as soon as you confirm that it is okay-ish :)

That's useful! LGTM

Looking at cudnn it actually looks fine already. I am merging this in as it.

Getting a build error

builder for '/nix/store/88l0mjgisahp4qrpymwm5nzal07iykdz-cudatoolkit-10.1.243.drv' failed with exit code 1; last 3 log lines:
  unpacking sources
  Creating directory pkg
  /nix/store/whpnlfj79vd8p175rp1bwg0x7wz5lmj1-stdenv-linux/setup: line 1301: cd: pkg/run_files: No such file or directory

Just saw that also. All the conditionals on the cuda-version were removed from the common.nix file. I'm sorry that I didn't review this PR before.
This should probably be reverted. @andir Could you push a revert and then we'll look into fixing it up again?

Thanks. I am sorry. I just pushed a revert: 464ff0a

Will look into the issue now and open another PR.

Looks like I lost a few changes during the rebase. I have an updated branch and will open a PR once the local builds are successful.

Thank you! 👍