numtide / flake-utils

Pure Nix flake utility functions [maintainer=@zimbatm]
MIT License
1.14k stars 78 forks source link

self parameter for eachSystem/eachDefaultSystem #78

Open WizardUli opened 2 years ago

WizardUli commented 2 years ago

When you want to reference a package from another package, or a devShell, etc... inside eachSystem/eachDefaultSystem closure then you have to either use the pesky rec keyword which even does not work for all cases or you have to use self parameter from the flake itself like self.${system}.packages.abc which is unnecessarily repeating the system.

I propose adding a variant of eachSystem/eachDefaultSystem which could be used like this:

{
  outputs = { self, nixpkgs, flake-utils }:
    flake-utils.lib.eachDefaultSystem2 (sself: system: {
      packages.abc = ...;

      devShells.default = pkgs.mkShell {
        packages = [
          sself.packages.abc # Instead of self.${system}.packages.abc;
        ];
      };
    });
}

Perhaps it could be made backward compatible similar to overrideAttrs (https://github.com/NixOS/nixpkgs/pull/119942) so new variant of eachSystem may not be necessary.

zimbatm commented 2 years ago

That's a good observation. As long as it doesn't break backwards compatibility, I don't mind extending the existing functions. Otherwise, it's probably best to introduce a new one (if you're sending a PR).

nrdxp commented 1 year ago

fwiw, divnix/nosys does just what you are looking for. I did start #83 to maybe see if there was interest in bringing this functionality to live here.