Skip to content

Commit

Permalink
Revert nixFlake / nixExperimental (NixOS#120141)
Browse files Browse the repository at this point in the history
Usage of the nixFlakes attribute obscures the fact that it's an
experimental feature. Providing a nixExperimental attribute (which
people will inevitably start using on their production machines) makes
it too easy to enable all experimental features, when you should
explicitly opt in to the features that you want to try out.

Also, upstream Nix doesn't provide an "enable all experimental
features" patch so neither should Nixpkgs.
  • Loading branch information
edolstra committed May 21, 2021
1 parent 224df69 commit 21bfd3b
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 37 deletions.
8 changes: 1 addition & 7 deletions pkgs/tools/package-management/nix/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -227,12 +227,6 @@ in rec {
];
});

nixExperimental = nixUnstable.overrideAttrs (prev: {
patches = (prev.patches or []) ++ [ ./enable-all-experimental.patch ];
});

nixFlakes = nixUnstable.overrideAttrs (prev: {
patches = (prev.patches or []) ++ [ ./enable-flakes.patch ];
});
nixFlakes = nixUnstable;

}
14 changes: 0 additions & 14 deletions pkgs/tools/package-management/nix/enable-all-experimental.patch

This file was deleted.

14 changes: 0 additions & 14 deletions pkgs/tools/package-management/nix/enable-flakes.patch

This file was deleted.

3 changes: 1 addition & 2 deletions pkgs/top-level/all-packages.nix
Original file line number Diff line number Diff line change
Expand Up @@ -30322,8 +30322,7 @@ in
nix
nixStable
nixUnstable
nixFlakes
nixExperimental;
nixFlakes;

nixStatic = pkgsStatic.nix;

Expand Down

0 comments on commit 21bfd3b

Please sign in to comment.