0
@@ -1,5 +1,7 @@
0
 *unreleased*
0
 
0
+* If :mode is given to upload() helper, do a chmod after to set the mode [Jamis Buck]
0
+
0
 * Fix load_from_file method for windows users [Neil Wilson]
0
 
0
 * Display a deprecation error if a remote git branch is specified [Tim Harper]

0
@@ -10,7 +10,6 @@ module Capistrano
0
         # set the mode on the file.
0
         def put(data, path, options={})
0
           opts = options.dup
0
-          opts[:permissions] = opts.delete(:mode)
0
           upload(StringIO.new(data), path, opts)
0
         end
0
     
0
@@ -23,7 +22,12 @@ module Capistrano
0
         end
0
 
0
         def upload(from, to, options={}, &block)
0
+          mode = options.delete(:mode)
0
           transfer(:up, from, to, options, &block)
0
+          if mode
0
+            mode = mode.is_a?(Numeric) ? mode.to_s(8) : mode.to_s
0
+            run "chmod #{mode} #{to}"
0
+          end
0
         end
0
 
0
         def download(from, to, options={}, &block)

0
@@ -14,7 +14,8 @@ class ConfigurationActionsFileTransferTest < Test::Unit::TestCase
0
 
0
   def test_put_should_delegate_to_upload
0
     @config.expects(:upload).with { |from, to, opts|
0
-      from.string == "some data" && to == "test.txt" && opts == { :permissions => 0777 } }
0
+      from.string == "some data" && to == "test.txt" && opts == { :mode => 0777 } }
0
+    @config.expects(:run).never
0
     @config.put("some data", "test.txt", :mode => 0777)
0
   end
0
 
0
@@ -28,6 +29,24 @@ class ConfigurationActionsFileTransferTest < Test::Unit::TestCase
0
     @config.upload("testl.txt", "testr.txt", :foo => "bar")
0
   end
0
 
0
+  def test_upload_without_mode_should_not_try_to_chmod
0
+    @config.expects(:transfer).with(:up, "testl.txt", "testr.txt", :foo => "bar")
0
+    @config.expects(:run).never
0
+    @config.upload("testl.txt", "testr.txt", :foo => "bar")
0
+  end
0
+
0
+  def test_upload_with_mode_should_try_to_chmod
0
+    @config.expects(:transfer).with(:up, "testl.txt", "testr.txt", :foo => "bar")
0
+    @config.expects(:run).with("chmod 775 testr.txt")
0
+    @config.upload("testl.txt", "testr.txt", :mode => 0775, :foo => "bar")
0
+  end
0
+
0
+  def test_upload_with_symbolic_mode_should_try_to_chmod
0
+    @config.expects(:transfer).with(:up, "testl.txt", "testr.txt", :foo => "bar")
0
+    @config.expects(:run).with("chmod g+w testr.txt")
0
+    @config.upload("testl.txt", "testr.txt", :mode => "g+w", :foo => "bar")
0
+  end
0
+
0
   def test_download_should_delegate_to_transfer
0
     @config.expects(:transfer).with(:down, "testr.txt", "testl.txt", :foo => "bar")
0
     @config.download("testr.txt", "testl.txt", :foo => "bar")