New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove manual service #83199
Remove manual service #83199
Conversation
Sounds good to me in principle. Let's add |
I think having the manual already open on the installer is useful and cool. Also rogue is nice when you have to wait for I have a happy memory of the first time I installed NixOS: I was surprised to discover the non-graphical installer was so friendly compared to arch linux. |
Maybe making the modules internal and only enabling them on the installer could be a solution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I like this approach and IMO we don't loose any functionality (regarding the default configuration) while removing unnecessary complexity.
Adding a nice error message for the removed modules would improve the user experience, but I consider it optional as there should be very few users (but in any case it would probably be nice to have).
Running the manual on a TTY is useless in the graphical ISOs and not particularly useful in non-graphical ISOs (since you can also run 'nixos-help'). Fixes NixOS#83157.
67c81af
to
bd379be
Compare
The nixos-manual service was removed in NixOS#83199 but we missed the check in this test which prevents the "tested" job from succeeding [0][1]. [0]: https://hydra.nixos.org/build/115472896 [1]: https://hydra.nixos.org/build/115472896/nixlog/94/tail
This needs to use |
Like this? #83651 EDIT: I assume we will pick all the related changes to 20.03. |
Motivation for this change
This removes the
nixos-manual
service on TTY-8 since it isn't very useful (you can just runnixos-help
) and is a security issue if people enable it in regular systems.While I was at it, I also removed the
rogue
service (which has been disabled in the installer for a while).Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)