capistrano 2.5.14 → 2.5.15

data/{CHANGELOG.rdoc → CHANGELOG}

@@ -1,3 +1,11 @@
+== 2.5.15 / 14 Febuary 2010
+
+Fixes a feature request not to overwrite roles when using the ROLES environmental variable.
+
+  * #126 - The option to not overwriting the roles which are defined in the task definition.
+  * Removed the `upgrade` file as it has been a couple of years since 1.x was in the wild.
+  * Slight internal re-factor of the way we calculate the `version`
+
 == 2.5.14 / 18 January 2010
 
 Fixes a low-value bug, thanks to Chris G for the well submitted patch:

data/Rakefile

@@ -5,6 +5,7 @@ begin
   Jeweler::Tasks.new do |gem|
     gem.version
     gem.name            = "capistrano"
+    gem.executables     = %W(capify cap)
     gem.summary         = %Q{Capistrano – Welcome to easy deployment with Ruby over SSH}
     gem.description     = %Q{Capistrano is a utility and framework for executing commands in parallel on multiple remote machines, via SSH.}
     gem.homepage        = "http://github.com/capistrano/capistrano"

data/VERSION

@@ -1 +1 @@
-2.5.14
+2.5.15

data/lib/capistrano/cli/execute.rb

@@ -24,6 +24,7 @@ module Capistrano
         config = instantiate_configuration(options)
         config.debug = options[:debug]
         config.dry_run = options[:dry_run]
+        config.preserve_roles = options[:preserve_roles]
         config.logger.level = options[:verbose]
 
         set_pre_vars(config)

data/lib/capistrano/cli/options.rb

@@ -79,6 +79,10 @@ module Capistrano
             "Make the output as quiet as possible."
           ) { options[:verbose] = 0 }
 
+          opts.on("-r", "--preserve-roles",
+            "Preserve task roles"
+          ) { options[:preserve_roles] = true }
+
           opts.on("-S", "--set-before NAME=VALUE",
             "Set a variable before the recipes are loaded."
           ) do |pair|
@@ -112,7 +116,7 @@ module Capistrano
             "Display the Capistrano version, and exit."
           ) do
             require 'capistrano/version'
-            puts "Capistrano v#{Capistrano::Version::STRING}"
+            puts "Capistrano v#{Capistrano::Version}"
             exit
           end
 

data/lib/capistrano/configuration.rb

@@ -19,11 +19,12 @@ module Capistrano
   # define roles, and set configuration variables.
   class Configuration
     # The logger instance defined for this configuration.
-    attr_accessor :debug, :logger, :dry_run
+    attr_accessor :debug, :logger, :dry_run, :preserve_roles
 
     def initialize(options={}) #:nodoc:
       @debug = false
       @dry_run = false
+      @preserve_roles = false
       @logger = Logger.new(options)
     end
 

data/lib/capistrano/configuration/servers.rb

@@ -48,7 +48,9 @@ module Capistrano
             filter_server_list(hosts.uniq)
           end
         else
-          roles  = role_list_from(ENV['ROLES'] || options[:roles] || self.roles.keys)
+					roles = role_list_from(ENV['ROLES'] || options[:roles] || self.roles.keys)
+					roles = roles & Array(options[:roles]) if preserve_roles && !options[:roles].nil?
+
           only   = options[:only] || {}
           except = options[:except] || {}
           
@@ -93,4 +95,4 @@ module Capistrano
       end
     end
   end
-end
+end

data/lib/capistrano/recipes/deploy.rb

@@ -45,14 +45,13 @@ _cset :version_dir,       "releases"
 _cset :shared_dir,        "shared"
 _cset :shared_children,   %w(system log pids)
 _cset :current_dir,       "current"
+_cset :restart_method     "apache"
 
 _cset(:releases_path)     { File.join(deploy_to, version_dir) }
 _cset(:shared_path)       { File.join(deploy_to, shared_dir) }
 _cset(:current_path)      { File.join(deploy_to, current_dir) }
 _cset(:release_path)      { File.join(releases_path, release_name) }
 
-# re: https://capistrano.lighthouseapp.com/projects/8716/tickets/88-getting-the-newest-directory
-# remove system inconsistencies with ls and let ruby sort the releases
 _cset(:releases)          { capture("ls -x #{releases_path}").split.sort }
 _cset(:current_release)   { File.join(releases_path, releases.last) }
 _cset(:previous_release)  { releases.length > 1 ? File.join(releases_path, releases[-2]) : nil }

data/lib/capistrano/version.rb

@@ -1,18 +1,15 @@
-require 'net/ssh/version'
-
 module Capistrano
 
-  # Describes the current version of Capistrano.
-  class Version < Net::SSH::Version
-    MAJOR = 2
-    MINOR = 5
-    TINY  = 14
+  class Version
+
+    CURRENT = File.read(File.dirname(__FILE__) + '/../../VERSION')
 
-    # The current version, as a Version instance
-    CURRENT = new(MAJOR, MINOR, TINY)
+    STRING = CURRENT.to_s
 
-    # The current version, as a String instance
-    STRING  = CURRENT.to_s
+    def self.to_s
+      CURRENT
+    end
+    
   end
 
 end

data/test/cli/execute_test.rb

@@ -15,7 +15,7 @@ class CLIExecuteTest < Test::Unit::TestCase
   def setup
     @cli = MockCLI.new
     @logger = stub_everything
-    @config = stub(:logger => @logger, :debug= => nil, :dry_run= => nil)
+    @config = stub(:logger => @logger, :debug= => nil, :dry_run= => nil, :preserve_roles= => nil)
     @config.stubs(:set)
     @config.stubs(:load)
     @config.stubs(:trigger)
@@ -129,4 +129,4 @@ class CLIExecuteTest < Test::Unit::TestCase
     MockCLI.expects(:parse).with(ARGV).returns(cli)
     MockCLI.execute
   end
-end
+end

data/test/cli/options_test.rb

@@ -93,6 +93,18 @@ class CLIOptionsTest < Test::Unit::TestCase
     assert_equal 0, @cli.options[:verbose]
   end
 
+  def test_parse_options_with_r_should_set_preserve_roles_option
+    @cli.args << "-r"
+    @cli.parse_options!
+    assert @cli.options[:preserve_roles]
+  end
+
+  def test_parse_options_with_preserve_roles_should_set_preserve_roles_option
+    @cli.args << "--preserve-roles"
+    @cli.parse_options!
+    assert @cli.options[:preserve_roles]
+  end
+
   def test_parse_options_with_S_should_set_pre_vars
     @cli.args << "-S" << "foo=bar"
     @cli.parse_options!

data/test/configuration/actions/invocation_test.rb

@@ -6,6 +6,7 @@ class ConfigurationActionsInvocationTest < Test::Unit::TestCase
     attr_reader :options
     attr_accessor :debug
     attr_accessor :dry_run
+		attr_accessor :preserve_roles
 
     def initialize
       @options = {}

data/test/configuration/servers_test.rb

@@ -5,9 +5,11 @@ require 'capistrano/configuration/servers'
 class ConfigurationServersTest < Test::Unit::TestCase
   class MockConfig
     attr_reader :roles
+		attr_accessor :preserve_roles
 
     def initialize
       @roles = {}
+			@preserve_roles = false
     end
 
     include Capistrano::Configuration::Servers
@@ -62,6 +64,33 @@ class ConfigurationServersTest < Test::Unit::TestCase
     ENV.delete('ROLES')
   end
 
+  def test_task_with_roles_as_environment_variable_and_preserve_roles_should_apply_only_to_existant_task_role
+    ENV['ROLES'] = "app,file"
+		@config.preserve_roles = true
+    task = new_task(:testing,@config, :roles => :app)
+    assert_equal %w(app1 app2 app3).sort, @config.find_servers_for_task(task).map { |s| s.host }.sort
+  ensure
+    ENV.delete('ROLES')
+	end
+
+  def test_task_with_roles_as_environment_variable_and_preserve_roles_should_apply_only_to_existant_task_roles
+    ENV['ROLES'] = "app,file,web"
+		@config.preserve_roles = true
+    task = new_task(:testing,@config, :roles => [ :app,:file ])
+    assert_equal %w(app1 app2 app3 file).sort, @config.find_servers_for_task(task).map { |s| s.host }.sort
+  ensure
+    ENV.delete('ROLES')
+	end
+
+  def test_task_with_roles_as_environment_variable_and_preserve_roles_should_not_apply_if_not_exists_those_task_roles
+    ENV['ROLES'] = "file,web"
+		@config.preserve_roles = true
+    task = new_task(:testing,@config, :roles => [ :app ])
+    assert_equal [], @config.find_servers_for_task(task).map { |s| s.host }.sort
+  ensure
+    ENV.delete('ROLES')
+	end
+	
   def test_task_with_hosts_as_environment_variable_should_apply_only_to_those_hosts
     ENV['HOSTS'] = "foo,bar"
     task = new_task(:testing)
@@ -126,4 +155,4 @@ class ConfigurationServersTest < Test::Unit::TestCase
     assert_equal %w(app1 app2 app3), @config.find_servers(:roles => lambda { :app }).map { |s| s.host }.sort
     assert_equal %w(app2 file), @config.find_servers(:roles => lambda { [:report, :file] }).map { |s| s.host }.sort
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: capistrano
 version: !ruby/object:Gem::Version 
-  version: 2.5.14
+  version: 2.5.15
 platform: ruby
 authors: 
 - Jamis Buck
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-01-19 00:00:00 +01:00
+date: 2010-02-14 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -78,18 +78,17 @@ email:
 - jamis@jamisbuck.org
 - lee.hambley@gmail.com
 executables: 
-- cap
 - capify
+- cap
 extensions: []
 
 extra_rdoc_files: 
-- README.rdoc
+- README
 files: 
 - .gitignore
-- CHANGELOG.rdoc
-- README.rdoc
+- CHANGELOG
+- README
 - Rakefile
-- TESTING.md
 - VERSION
 - bin/cap
 - bin/capify
@@ -144,7 +143,6 @@ files:
 - lib/capistrano/recipes/deploy/templates/maintenance.rhtml
 - lib/capistrano/recipes/standard.rb
 - lib/capistrano/recipes/templates/maintenance.rhtml
-- lib/capistrano/recipes/upgrade.rb
 - lib/capistrano/role.rb
 - lib/capistrano/server_definition.rb
 - lib/capistrano/shell.rb

data/TESTING.md

@@ -1,16 +0,0 @@
-Testing Capistrano
-==================
-
- 1. Install the Following Gem Dependencies
- 
-    `# gem install mocha echoe`
-
- 2. Run the following
-    
-    `$ rake test`
-    
- 3. Output Should Resemble
- 
-    `Finished in 0.859017 seconds.`
-    `523 tests, 1271 assertions, 0 failures, 0 errors`
-  

data/lib/capistrano/recipes/upgrade.rb

@@ -1,33 +0,0 @@
-# Tasks to aid the migration of an established Capistrano 1.x installation to
-# Capistrano 2.x.
-
-namespace :upgrade do
-  desc <<-DESC
-    Migrate from the revisions log to REVISION. Capistrano 1.x recorded each \
-    deployment to a revisions.log file. Capistrano 2.x is cleaner, and just \
-    puts a REVISION file in the root of the deployed revision. This task \
-    migrates from the revisions.log used in Capistrano 1.x, to the REVISION \
-    tag file used in Capistrano 2.x. It is non-destructive and may be safely \
-    run any number of times.
-  DESC
-  task :revisions, :except => { :no_release => true } do
-    revisions = capture("cat #{deploy_to}/revisions.log")
-
-    mapping = {}
-    revisions.each do |line|
-      revision, directory = line.chomp.split[-2,2]
-      mapping[directory] = revision
-    end
-
-    commands = mapping.keys.map do |directory|
-      "echo '.'; test -d #{directory} && echo '#{mapping[directory]}' > #{directory}/REVISION"
-    end
-
-    command = commands.join(";")
-
-    run "cd #{releases_path}; #{command}; true" do |ch, stream, out|
-      STDOUT.print(".")
-      STDOUT.flush
-    end
-  end
-end