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
doc/stdenv: put note about remote builds with breakpointHook into a <note> #61049
Conversation
c4371d4
to
23f3b4b
Compare
@worldofpeace as this applies to |
Yeah, I think typically docs changes are good to backport if possible. |
23f3b4b
to
f415573
Compare
Oh damn it, that also means that I overlooked it when I was reading the docs in the first place 😅 |
f415573
to
3ce9a1f
Compare
…note> With remote builds, the sandbox can't be accessed by `cntr` as it is on a different machine. I decided to put this into an extra `note` block as it took me admittedly too much time to figure this out.
3ce9a1f
to
0f9653f
Compare
fixed. And also modified the commit message accordingly. |
Understandable, I almost missed it also. Plus with this we can tell how it disable it. |
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.
LGTM, everything was addressed that could be.
Motivation for this change
With remote builds, the sandbox can't be accessed by
cntr
as it is ona different machine. I decided to document this in the manual as it took
me admittedly too much time to figure this out.
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)