Skip to content

Replace references to all-packages.nix, by references to the top-level#13896

Merged
edolstra merged 1 commit intoNixOS:masterfrom
nbp:import-top-level
Mar 14, 2016
Merged

Replace references to all-packages.nix, by references to the top-level#13896
edolstra merged 1 commit intoNixOS:masterfrom
nbp:import-top-level

Conversation

@nbp
Copy link
Member

@nbp nbp commented Mar 13, 2016

This change is extracted from the larger pull request #10851 , and was requested as a cleaner approach than was was done in #10307.

This change is useful, as we now have only a single point of entry in all-packages.nix, which is the top-level default.nix file of nixpkgs.

Having this change, will allow to split all-packages.nix file into one which contains the logic, and another one which contains the list of packages. Thus giving us the ability to re-indent this one correctly.

cc @domenkozar

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @copumpkin, @edolstra and @pikajude to be potential reviewers

@nbp nbp added 0.kind: enhancement Add something new or improve an existing system. 6.topic: closure size The final size of a derivation, including its dependencies 8.has: clean-up This PR removes packages or removes other cruft and removed 6.topic: closure size The final size of a derivation, including its dependencies labels Mar 13, 2016
@nbp nbp added this to the 16.03 milestone Mar 13, 2016
@nbp nbp self-assigned this Mar 13, 2016
@nbp nbp modified the milestones: 16.09, 16.03 Mar 13, 2016
edolstra added a commit that referenced this pull request Mar 14, 2016
Replace references to all-packages.nix, by references to the top-level
@edolstra edolstra merged commit 27af419 into NixOS:master Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0.kind: enhancement Add something new or improve an existing system. 8.has: clean-up This PR removes packages or removes other cruft

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants