capistrano / rvm

MIT License
140 stars 47 forks source link

Remove SSHKit dependency #31

Closed Kriechi closed 10 years ago

Kriechi commented 10 years ago

capistrano-rvm blocks the usage of a newer SSHKit gem version, because of the over restrictive version selector.

I think we should remove the SSHKit dependency completely.

mpapis commented 10 years ago

I would change it to >= as it was required for the prefix on command map

Kriechi commented 10 years ago

Good catch. Thanks. I didn't think about the command map. Of course, then i would suggest using >=1.2.

kirs commented 10 years ago

We have SSHKit dependency because it's used inside the capistrano-rbenv code: https://github.com/capistrano/rvm/blob/master/lib/capistrano/tasks/rvm.rake#L35

It's important to have >=1.2 because we need exact features from this version.

capistrano-rvm blocks the usage of a newer SSHKit gem version, because of the over restrictive version selector. that means we just have to use flexible dependency, right?

kirs commented 10 years ago

Fixed: https://github.com/capistrano/rvm/blob/master/capistrano-rvm.gemspec

mpapis commented 10 years ago

I still see:

gem.add_dependency 'sshkit', '~> 1.2'

was it supposed to be >=?

kirs commented 10 years ago

It wasn't, because >= would allow to use 2.x that may have breaking API changes.

mpapis commented 10 years ago

then I'm confused, 1.2 introduced that breaking changes, it should be pushed to 2.0 then ... just saying