Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@henderea
Copy link
Contributor

some commands (like rvm list known) display error messages like command not found: __rvm_remote_version or __rvm_cli_autoupdate_version_old:2: command not found: __rvm_remote_version

Looking into code, the actual function name is __rvm_version_remote, so this commit fixes that reference. I only found the bad reference in a single location.

It doesn't seem to cause any significant harm, but fixing this will clean up an error message that appears in the terminal when some commands are run.

I didn't find an existing github issue for this.

Changes proposed in this pull request:

  • Fix a function reference to eliminate an error message

Make sure that your pull request includes entry in the CHANGELOG.

henderea added 2 commits June 30, 2017 11:34
some commands (like `rvm list known`) display error messages like `command not found: __rvm_remote_version` or `__rvm_cli_autoupdate_version_old:2: command not found: __rvm_remote_version`

Looking into code, the actual function name is `__rvm_version_remote`, so this commit fixes that reference.  I only found the bad reference in a single location.
@pkuczynski pkuczynski changed the title fix an error message displayed on some commands Command not found __rvm_remote_version error Jun 30, 2017
@pkuczynski
Copy link
Member

Aaaaaaaa, what a silly bug if mine! Thanks! :)

@pkuczynski pkuczynski merged commit aacfb27 into rvm:master Jun 30, 2017
@pkuczynski pkuczynski added this to the rvm-1.29.3 milestone Jun 30, 2017
@pkuczynski pkuczynski self-requested a review June 30, 2017 15:44
@henderea
Copy link
Contributor Author

Wow, talk about fast. Less than 10 minutes after my original commit and it's already merged. Must have caught you when you were online 😄

Thanks.

@pkuczynski
Copy link
Member

I am always online :) Just not always able to review PR :)

henderea added a commit to henderea/rvm that referenced this pull request Jun 30, 2017
henderea added a commit to henderea/rvm that referenced this pull request Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants