perSystem: add a default value for systems
#28
Conversation
There was a problem hiding this comment.
After some discussion and researching the flakeExposed attribute, I don't think we should have this default here. Hang tight for a solution though.
The flakeExposed attribute is a very recent addition to Nixpkgs, which makes me a bit uneasy about the stability of this (while acknowledging that flakes are entirely unstable).
Setting this default also goes against how Eelco intended systems to be used: to provide known-working attributes. By setting this default, we remove the "known" aspect and more likely than not, the "working" aspect. That would be bad for your users and all flake-parts users.
flake-parts is intended to be a common core for many distributed expressions to interoperate. Bringing an opinionated default into the core defeats its purpose as a unifying interface, as it gives others reasons to reject it.
As an example, I've only accepted the default pkgs because I expect almost everyone to agree on its definition and the existence of a pkgs module argument being a unifying force, until a better solution is applied (nixpkgs.flakeModules.pkgs, TBD).
But, while I'm sorry for the bad news part, there is a solution, which is to create your own module of opinionated defaults. If this single line is really important, presumably you have it in more than one repo, so it might make sense to create a module with personal defaults in a separate repo. You can use that not just to set the default systems, but also a default Nix formatter with pre-commit-hooks-nix for example.
|
@roberth No worries.
Is there documentation on 'creating modules for flake-parts'? The linked PR has |
No docs for writing flake-parts modules specifically yet, but the general structure like Instead of NixOS options like
When you write a simple module, it's often possible to omit {
systems = ["x86_64-linux"];
}{
config.systems = ["x86_64-linux"];
}This is also part of the reason why you don't have to use |
Resolves #25 - specifically #25 (comment)