treewide: replace broken udev paths with systemd#268718
Conversation
|
Does the eudev udevadm work with the systemd udevd? |
|
@NickCao to the best of my knowledge, yes; it aims to be independent of the init system while maintaining compatibility. Perhaps we should find/add some NixOS tests for this. |
Still it feels strange that some udevadms are from systemd while others are from eudev, how about just use systemd, is the impact on closure size that big? |
|
cc @NixOS/systemd Also I believe the |
|
We can use |
Maybe not #264163 And that PR suggests:
So simply systemd is the choice? |
|
@NickCao excellent point. On closer look, I discovered that all instances where I initially used eudev, already had systemd as a dependency. |
|
@ofborg test hardened |
|
@ofborg test stratis |
|
@ofborg test os-prober |
|
Result of 1 package blacklisted:
6 packages built:
|
Description of changes
Currently, modules like
usmbuxdare broken because they use an old path to/bin/udevadm. This PR fixes all instances of${pkgs.udev}/bin/udevadmwith the following rules:${config.systemd.package}/bin/udevadm${pkgs.systemd}/bin/udevadmThings done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)