From 22791083dab2de2b22154dd4e0e593ff6c48e08b Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Thu, 31 Jan 2013 04:13:07 -0600 Subject: Fix -in behavior: descend into existing directory without prompting, show full path in error messages, actually overwrite when answering yes to -i. --- toys/posix/cp.c | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) (limited to 'toys/posix') diff --git a/toys/posix/cp.c b/toys/posix/cp.c index 94b134aa..705c4bfb 100644 --- a/toys/posix/cp.c +++ b/toys/posix/cp.c @@ -85,8 +85,19 @@ int cp_node(struct dirtree *try) // Handle -inv - if ((flags & (FLAG_i|FLAG_n)) && !faccessat(cfd, catch, R_OK, 0)) - if ((flags & FLAG_n) || !yesno("cp: overwrite", 1)) return 0; + if (!faccessat(cfd, catch, F_OK, 0) && !S_ISDIR(cst.st_mode)) { + char *s; + + if (S_ISDIR(try->st.st_dev)) { + error_msg("dir at '%s'", s = dirtree_path(try, 0)); + free(s); + } else if (flags & FLAG_n) return 0; + else if (flags & FLAG_i) { + fprintf(stderr, "cp: overwrite '%s'", s = dirtree_path(try, 0)); + free(s); + if (!yesno("", 1)) return 0; + } + } if (flags & FLAG_v) { char *s = dirtree_path(try, 0); @@ -160,7 +171,7 @@ int cp_node(struct dirtree *try) close(fdin); } } - } while (err && (flags & FLAG_f) && !unlinkat(cfd, catch, 0)); + } while (err && (flags & (FLAG_f|FLAG_n)) && !unlinkat(cfd, catch, 0)); } if (fdout != -1) { -- cgit v1.2.3