Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.Sign up
nixos/systemd: Use a proper type for unit paths #75510
Motivation for this change
Idk if this breaks anything, and I also have no idea which tests to run on ofborg for that…
cc @edolstra because you wrote the original code
Thank you for your contributions.
This has been automatically marked as stale because it has had no activity for 180 days.
If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity.
Here are suggestions that might help resolve this more quickly:
Hm, I only meant to edit my own comment, which suggested something silly, not delete the whole comment thread, sorry. Also it seems github swallowed my answer as well :-/
I'm not sure if there isn't anything setting this option to a static path, but in that case, we can probably change this be a list of str or package - I think only pointing it to packages is fine, and I want to know of obscure usecases where it needs to be different ;-)
Turns out, NixOS#75510 was too restrictive. We also need to allow str here, as some modules set this to "/run/wrappers" to bring `/run/wrappers/bin` into $PATH of a unit.