Skip to content
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

functions: nvm.fish: don't attempt to overwrite $fish_user_paths #121

Conversation

jackwestmoretab
Copy link

$fish_user_paths is set by the shell at startup, and should not be
overwritten with the -U flag. Doing so results in an error.

Instead, drop the -U flag, and simply append the NVM version bin to the
end of the user's $fish_user_paths.

$fish_user_paths is set by the shell at startup, and should not be
overwritten with the -U flag. Doing so results in an error.

Instead, drop the -U flag, and simply append the NVM version bin to the
end of the user's $fish_user_paths.
@jorgebucaran
Copy link
Owner

jorgebucaran commented Nov 14, 2020

@jackwestmoretab Isn't this the same as #107? It isn't. My bad. 😂

@jackwestmoretab
Copy link
Author

@jorgebucaran having reviewed #107, I think that the suggestion in #107 (comment) is the right way to go i.e. modify $PATH directly, rather than using $fish_user_paths (for the same reasons given in #107 (comment)).

@jackwestmoretab
Copy link
Author

To be clear, I think that the command should manipulate $PATH directly (rather than use set -g ... as the changes in #107 propose).

@jackwestmoretab
Copy link
Author

p.s. thanks for linking in #101. I'll close this, as I think proposals there, and in #107, are superior to this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants