Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Prevent copying a directory into itself #4

Open
AvianFlu opened this issue Sep 28, 2011 · 1 comment
Open

Prevent copying a directory into itself #4

AvianFlu opened this issue Sep 28, 2011 · 1 comment

Comments

@AvianFlu
Copy link
Owner

cp -r stops users from doing this. ncp currently continues to recurse, resulting eventually in ENAMETOOLONG after many levels of directory nesting. While an obscure situation, this should be fixed.
@anandkumarpatel
Copy link

repro here:
http://runnable.com/U2vWs4L1zLJeDQ-Q/ncp-recursive-copy-issue

This was referenced Sep 9, 2014
Closed
Closed
AvianFlu pushed a commit that referenced this issue Sep 24, 2014
@iwege @AvianFlu
Prevent copying a directory into itself. Links to #4
73eca73 
Conflicts:
	test/ncp.js
@iwege iwege mentioned this issue Sep 25, 2014
Open
AvianFlu added a commit that referenced this issue Oct 2, 2014
@AvianFlu
Revert "Prevent copying a directory into itself. Links to #4"
1b94da1 
This reverts commit 73eca73.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AvianFlu @anandkumarpatel