Fixes for Capistrano requirement when using vlad & Rake::FetchError #892

Closed
wants to merge 1 commit into
from

5 participants

@ecoleman

Sometime between 1.0.3 and 1.0.7 vlad deployment broke. Added a check for "Capistrano" and resolved the FetchError by defaulting :default_roles to false.

@ecoleman ecoleman Fixes:
 Undefined Capistrano constant when using vlad
 Rake::FetchError on :default_roles
9376add
@cdemyanovich

Would love to see this in 1.0.8, since I was just looking into this, too. I was looking for a way to spec this behavior, something along the lines of ensuring nothing is raised if we require 'bundler/vlad' wen capistrano is not installed. I haven't verified, though, that having capistrano installed avoids the problem.

@cdemyanovich

By the way, accepting this pull request would also close #884.

@sl4m

I would like to see this pulled as well. This is a showstopper for our project.

@fpauser

Need this too!

@ecoleman

Allow bundler/vlad to work in vlad deploy scripts

Fixes:
Undefined Capistrano constant when using vlad
Rake::FetchError on :default_roles

Closed by 2bbc732, #903

@leobessa leobessa pushed a commit that referenced this pull request Apr 18, 2011
@ecoleman ecoleman Allow bundler/vlad to work in vlad deploy scripts
Fixes:
 Undefined Capistrano constant when using vlad
 Rake::FetchError on :default_roles

Closes #892, #903
2bbc732
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment