Fix `rvm_path` detection in RVM mixed mode. #54

Merged
merged 2 commits into from May 29, 2012

Projects

None yet

2 participants

@mpapis
mpapis commented May 24, 2012

in RVM mixed mode, rvm and rubies are installed in system and gemsets are installed in user HOME, this was making the code to falsely detect $HOME/.rvm as rvm_path, while it should be /usr/local/rvm.

This change adds checking for $rvm_path/scripts/rvm which is available only in the isntallation path not in the mixed mode in HOME.

@FooBarWidget
Member

pathh - I think that's a typo.

@mpapis
mpapis commented May 24, 2012

ah yes, sorry for that, fixed

@FooBarWidget
Member

Thanks mpapis, this looks good. We require contributors to sign our contributor agreement before we can merge in their patches. Could you sign it at http://www.phusion.nl/forms/contributor_agreement?

@mpapis
mpapis commented May 28, 2012

email with agreement sent <3

@FooBarWidget FooBarWidget merged commit 8c9e147 into phusion:master May 29, 2012
@FooBarWidget
Member

Thanks Michal. :)

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