New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

symlinks need full dest path for FileUtils.copy_entry #863

Merged
merged 1 commit into from Feb 24, 2015

Conversation

Projects
None yet
2 participants
@fetep
Contributor

fetep commented Feb 24, 2015

From http://ruby-doc.org/stdlib-2.2.0/libdoc/fileutils/rdoc/FileUtils.html#method-c-copy_entry:

Both of src and dest must be a path name. src must exist, dest must not exist.

(tested & working)

@jordansissel

This comment has been minimized.

Owner

jordansissel commented Feb 24, 2015

I'd like to see tests for this, but we can do that later. Code LGTM.

jordansissel added a commit that referenced this pull request Feb 24, 2015

Merge pull request #863 from fetep/fix_dir_symlink
symlinks need full dest path for FileUtils.copy_entry

@jordansissel jordansissel merged commit e463e4c into jordansissel:master Feb 24, 2015

jordansissel added a commit that referenced this pull request Apr 24, 2015

Merge pull request #863 from fetep/fix_dir_symlink
symlinks need full dest path for FileUtils.copy_entry

jordansissel added a commit that referenced this pull request Jun 20, 2016

Merge pull request #863 from fetep/fix_dir_symlink
symlinks need full dest path for FileUtils.copy_entry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment