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

added rbenv-check. checks rbenv installed and configured correctly #377

Closed
wants to merge 2 commits into from
Closed

added rbenv-check. checks rbenv installed and configured correctly #377

wants to merge 2 commits into from

Conversation

dyladan
Copy link

@dyladan dyladan commented Apr 11, 2013

Specifically it checks that:

  1. rbenv bin directory is in path
  2. rbenv in bin directory is executable
  3. rbenv shim directory is in path
  4. which ruby returns the shim ruby and not system ruby

It outputs in convenient color coded format that draws attention to problems.

@mislav
Copy link
Member

mislav commented Apr 11, 2013

Looks useful. Compare to the "doctor" script I created for the wiki.

However, what sense does it make for this to be an rbenv subcommand if you can't execute it if rbenv isn't in your PATH? The instructions for the command should then say to execute it with full path, e.g. /path/to/rbenv/libexec/rbenv-check.

Also, colorization should be off is standard output is not to a tty.

Also, not sure if we'll pull this. I'll let @sstephenson weigh on this one

@dyladan
Copy link
Author

dyladan commented Apr 12, 2013

I created this because of a problem that I had with paths that was causing my to use the system ruby and it took me a few minutes before I ran which ruby and figured it out.

@dyladan dyladan closed this Apr 12, 2013
@dyladan
Copy link
Author

dyladan commented Apr 12, 2013

oops I didn't mean to close this.

@dyladan dyladan reopened this Apr 12, 2013
@mislav mislav added this to the 1.0 milestone Jan 13, 2015
@mislav mislav mentioned this pull request Oct 26, 2015
14 tasks
@mislav
Copy link
Member

mislav commented Dec 24, 2015

Closing in favor of official rbenv-doctor script that's being developed in a separate repo.

screen shot 2015-12-24 at 1 15 19 am

Thanks for the initial suggestions! I've initially based the logic in my script on the checks proposed above.

@mislav mislav closed this Dec 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants