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
nix-daemon service: Ensure ssh
is on PATH. Fixes #46038.
#46041
Conversation
This fixes a regression introduced in commit 700e21d nix needs ssh on path for the SSH substituter functionality, not only the distributed builds functionality. Signed-off-by: Niklas Hambüchen <mail@nh2.me>
If merged, I think this should be backported to 18.09. |
LGTM.
|
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.
Looks okay to me. I'll give @edolstra a chance to respond if this doesn't look right. I know there has been some confusion in the past as what to include for Nix's dependencies.
Ideally this would be conditional on whether distributed builds are enabled or there are any ssh:// substituters. |
@edolstra I don't think this can be done since you don't necessarily have to have these in your config, you can also give them on the command line. Or am I wrong? |
Should the wiki be updated to reflect the merged status of this PR?
|
@aaronjanse I think so. |
This fixes a regression introduced in commit
700e21d
nix needs ssh on path for the SSH substituter functionality,
not only the distributed builds functionality.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)