search

a-chernykh / railsbox

Fast and easy Ruby on Rails virtual machines
https://railsbox.io
MIT License
320 stars 52 forks source link

Reversed order of check for RVM non-binary #31

Closed eavonius closed 8 years ago

eavonius commented 9 years ago

If you provision once with a failure but past this step, then provision again, line 71 will fail. Reversing the order of the when clause succeeds because item.rc will not equal zero but install_rubies results will be empty and skipped because the first part of the and check fails so the second statement doesnt evaluate.

a-chernykh commented 8 years ago

Thanks @eavonius. Merging it in.