Correct Nushell source command #4230
Conversation
Yethal
force-pushed
the
proper-nushell-support
branch
from
e36dd77 to
064cf6a
Compare
| source "{cargo_home}/env.nu" # For nushell | ||
| . "{cargo_home}/env" # For sh/bash/zsh/ash/dash/pdksh | ||
| source "{cargo_home}/env.fish" # For fish | ||
| source $"($nu.home-path)/.cargo/env.nu" # For nushell |
There was a problem hiding this comment.
Note that this is not necessary the $CARGO_HOME! It's just its default value...
There was a problem hiding this comment.
Is there a scenario where $CARGO_HOME is set to a value different than default on a system and a human user will actually read this prompt? Or is it something that's only overriden in CI and such?
There was a problem hiding this comment.
@Yethal A human user will actually read the prompt. We allow $CARGO_HOME to be set to something else beforehand, actually IIRC we have suggested in an earlier note (before the actual installation, see https://github.com/search?q=repo%3Arust-lang%2Frustup+%22This+can+be+modified+with+the%22&type=code) that if you want it to be elsewhere you can set that and return to the installer.
|
It looks like we need something like https://www.nushell.sh/cookbook/foreign_shell_scripts.html but I'm not so sure... cc @LittleJianCH |
Nushell uses
$env.HOMEinstead of$HOMEsyntax for environment variables. However sincesourceis a parser keyword it cannot access environment variables which is why it should use$nu.home-pathinsteadAdditionally, the original command will cause nushell to error out on next config evaluation