capistrano / rvm

MIT License
140 stars 47 forks source link

remove using wrappers, fix #13, fix #14 #16

Closed mpapis closed 10 years ago

mpapis commented 10 years ago

switch to detect the binaries at runtime using rvm ruby do binary - this eliminates the need to create wrappers which solves:

kirs commented 10 years ago

It's a nice feature and it simplifies RVM integration a lot, but I think we need more testing by users.

mpapis commented 10 years ago

I also think it needs more testing, but this is one of basic features of rvm so I do not expect problems,

it will also allow adding flags like --create and --install:

rvm ruby@gemset --create --install do binary
Kriechi commented 10 years ago

@mpapis could you please updated the readme according to your changes. (once it's tested enough and your proposed solution works)

Zeophlite commented 10 years ago

I'm using this PR in my production code, so +1 for pulling

It integrates well with capistrano/bundler/pull/6

Kriechi commented 10 years ago

:+1: for pulling

kirs commented 10 years ago

Thanks for testing! I've pushed 0.0.3 version of the gem and it's ready to use :blush: