Bug #7246

FileUtils.cp_r does not preserve attributes of directories

Added by Jan Wedekind over 2 years ago. Updated about 2 years ago.

[ruby-core:48603]
Status:Closed
Priority:Normal
Assignee:Yusuke Endoh
ruby -v:ruby 1.9.3p194 (2012-04-20 revision 35410) [x86_64-darwin11.3.0] Backport:

Description

(({FileUtils.cp_r})) does not preserve the file attributes of directories because the attributes are set before copying to the directory.
The following monkey patch fixes this behaviour. However the monkey patch relies on the contents of the source directory not changing during the copy operation.

(({require 'fileutils'

def copy_entry(src, dest, preserve = false, dereference_root = false, remove_destination = false)
Entry_.new(src, nil, dereference_root).traverse do |ent|
destent = Entry_.new(dest, ent.rel, false)
File.unlink destent.path if remove_destination && File.file?(destent.path)
ent.copy destent.path
end
Entry_.new(src, nil, dereference_root).postorder_traverse do |ent|
destent = Entry_.new(dest, ent.rel, false)
ent.copy_metadata destent.path if preserve
end
end
module_function :copy_entry

end}))

Associated revisions

Revision 39015
Added by Yusuke Endoh about 2 years ago

  • lib/fileutils.rb (copy_entry, wrap_traverse): preserve attributes of
    directories on FileUtils.cp_r. The fix was proposed by Jan
    Wedekind. [Bug #7246]

  • test/fileutils/test_fileutils.rb: add a test for above.

Revision 39015
Added by Yusuke Endoh about 2 years ago

  • lib/fileutils.rb (copy_entry, wrap_traverse): preserve attributes of
    directories on FileUtils.cp_r. The fix was proposed by Jan
    Wedekind. [Bug #7246]

  • test/fileutils/test_fileutils.rb: add a test for above.

History

#1 Updated by Jan Wedekind over 2 years ago

The following code might be a proper bugfix.

(({require 'fileutils'

module FileUtils

class Entry_

def wrap_traverse(pre, post)
  pre.call self
  if directory?
    entries.each do |ent|
      ent.wrap_traverse pre, post
    end
  end
  post.call self
end

end

def copy_entry(src, dest, preserve = false, dereference_root = false, remove_destination = false)
Entry_.new(src, nil, dereference_root).wrap_traverse(proc do |ent|
destent = Entry_.new(dest, ent.rel, false)
File.unlink destent.path if remove_destination && File.file?(destent.path)
ent.copy destent.path
end, proc do |ent|
destent = Entry_.new(dest, ent.rel, false)
ent.copy_metadata destent.path if preserve
end)
end
module_function :copy_entry

end}))

#2 Updated by Yusuke Endoh over 2 years ago

  • Status changed from Open to Assigned
  • Assignee set to Yusuke Endoh
  • Target version set to 2.0.0

Thanks, I'll import your fix unless there is objection.

Yusuke Endoh mame@tsg.ne.jp

#3 Updated by Yusuke Endoh about 2 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r39015.
Jan, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/fileutils.rb (copy_entry, wrap_traverse): preserve attributes of
    directories on FileUtils.cp_r. The fix was proposed by Jan
    Wedekind. [Bug #7246]

  • test/fileutils/test_fileutils.rb: add a test for above.

Also available in: Atom PDF