add support for rbenv #45

Merged
merged 1 commit into from Jun 22, 2012

Conversation

Projects
None yet
4 participants
@chrislerum
Contributor

chrislerum commented May 3, 2012

Just a harmless little 'source if it exists' line for rbenv users.

@cyphactor

This comment has been minimized.

Show comment
Hide comment
@cyphactor

cyphactor May 3, 2012

Owner

Thanks for the pull request @chrislerum. As soon as I can get around to testing it, assuming that all works well and it doesn't effect RVM usage I will merge it in. Based on reading the diff I don't foresee it causing any problems but I want to test it for sure before I merge it into master and it gets distributed to the world.

Thanks Again

Owner

cyphactor commented May 3, 2012

Thanks for the pull request @chrislerum. As soon as I can get around to testing it, assuming that all works well and it doesn't effect RVM usage I will merge it in. Based on reading the diff I don't foresee it causing any problems but I want to test it for sure before I merge it into master and it gets distributed to the world.

Thanks Again

@@ -9,6 +9,13 @@
# to assume that those two things exist in the provided
# project path.
+function load_rbenv(){
+ if [ -d $HOME/.rbenv ]; then

This comment has been minimized.

@sarahhodne

sarahhodne May 19, 2012

This should probably be if [ -d "$HOME/.rbenv" ]; then to support people with spaces in their $HOME.

@sarahhodne

sarahhodne May 19, 2012

This should probably be if [ -d "$HOME/.rbenv" ]; then to support people with spaces in their $HOME.

This comment has been minimized.

@cyphactor

cyphactor May 19, 2012

Owner

I agree

@jpfuentes2

This comment has been minimized.

Show comment
Hide comment

+1

cyphactor added a commit that referenced this pull request Jun 22, 2012

@cyphactor cyphactor merged commit 81884f8 into cyphactor:master Jun 22, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment