diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | lib/fileutils.rb | 2 | ||||
-rw-r--r-- | test/fileutils/test_fileutils.rb | 10 |
3 files changed, 17 insertions, 1 deletions
@@ -1,3 +1,9 @@ +Fri Mar 18 08:48:06 2011 Oleg Shaldybin <oleg.shaldybin@gmail.com> + + * lib/fileutils.rb (FileUtils::Entry_#copy_file): updated FileUtils.cp + to still copy file permissions when :preserve is false (as cp does + this even when -p isn't set). + Fri Mar 18 00:59:38 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> * ext/win32ole/extconf.rb (create_docfile): removed. should not diff --git a/lib/fileutils.rb b/lib/fileutils.rb index 9cd1a7cc76..f2a61970b4 100644 --- a/lib/fileutils.rb +++ b/lib/fileutils.rb @@ -1276,7 +1276,7 @@ module FileUtils def copy_file(dest) File.open(path()) do |s| - File.open(dest, 'wb') do |f| + File.open(dest, 'wb', s.stat.mode) do |f| IO.copy_stream(s, f) end end diff --git a/test/fileutils/test_fileutils.rb b/test/fileutils/test_fileutils.rb index 3ec8dce2d5..290ba21f0d 100644 --- a/test/fileutils/test_fileutils.rb +++ b/test/fileutils/test_fileutils.rb @@ -230,6 +230,16 @@ class TestFileUtils } end + def test_cp_preserve_permissions + bug4507 = '[ruby-core:35518]' + touch 'tmp/cptmp' + chmod 0755, 'tmp/cptmp' + cp 'tmp/cptmp', 'tmp/cptmp2' + assert_equal(File.stat('tmp/cptmp').mode, + File.stat('tmp/cptmp2').mode, + bug4507) + end + def test_cp_symlink touch 'tmp/cptmp' # src==dest (2) symlink and its target |