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
[WIP] Add hardware.enableAllUdevRules #69168
Conversation
These aren’t really package and instead should be moved to their respective nixos module.
This can be used to enable all of the udev-rules modules we provide. Makes it easier when you aren’t sure what hardware you will be having. Should all be simple text files with no extra dependencies.
These options can now be merged into one big udev rules module.
These are not necessarily useful to everyone. Better to leave them as is.
9989cf9
to
4e4953b
Compare
It's a bit weird, but I guess udev rules really don't cost much to install (right?), so this is probably fine. I also like how this removes those trivial udev modules. |
I read the option as "add all environment.systemPackages to services.udev.packages" :-) |
Yeah that's definitely a bit confusing. Some other name might be better. "enableExtraUdevRules" or something similar. |
Thank you for your contributions.
|
This is similar to enableAllFirmware, but for udev rules. Basically, try to put in all of the udev rules we know of, without adding any extra dependencies. This unifies a lot of modules and should make maintenence easier in the future.
Also remove some -udev-rules "packages" that aren't really packages.
Perhaps we should merge these into one rules directory and publish it as "nixos-udev-rules" or something like that?
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @