-
-
Notifications
You must be signed in to change notification settings - Fork 8k
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
When NVM_DIR is a symlink, nvm_ls
breaks
#617
Labels
bugs
Oh no, something's broken :-(
pull request wanted
This is a great way to contribute! Help us out :-D
testing
Stuff related to testing nvm itself.
Comments
ljharb
added
testing
Stuff related to testing nvm itself.
pull request wanted
This is a great way to contribute! Help us out :-D
labels
Apr 5, 2015
This seems to also be causing some of the issues incorrectly reported on #855. |
tornewuff
added a commit
to tornewuff/dotfiles
that referenced
this issue
Jun 18, 2016
Use realpath to get the actual location. See nvm-sh/nvm#617 for the bug.
Temporary workaround, in # you can use the real path instead of the command f you prefer
export NVM_DIR="$(realpath $HOME/.nvm)"
[ -s "$NVM_DIR/nvm.sh" ] && . "$NVM_DIR/nvm.sh" |
travi
added a commit
to travi/dotfiles
that referenced
this issue
Jan 30, 2018
based on the workaround mentioned here: nvm-sh/nvm#617 (comment)
compilenix
added a commit
to compilenix/dotfiles
that referenced
this issue
Jun 14, 2018
This was referenced May 8, 2019
Closed
This was referenced Sep 13, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bugs
Oh no, something's broken :-(
pull request wanted
This is a great way to contribute! Help us out :-D
testing
Stuff related to testing nvm itself.
Per some of the discussion on here: #616 (comment) as reported by @patrick-steele-idem
Solution is to add
-L
to thefind
command insidenvm_ls
. The tricky part is writing a test for it :-)The text was updated successfully, but these errors were encountered: