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
nixos/systemd: Use a proper type for unit paths #75510
Conversation
@GrahamcOfBorg test systemd Uses |
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:
|
Bruh. |
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 ;-) |
I got a pointer on IRC that some modules add Will file a PR shortly. |
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.
Motivation for this change
Idk if this breaks anything, and I also have no idea which tests to run on ofborg for that…
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @edolstra because you wrote the original code