sbase

commit 8dfcbf23b0b767f941d55364d8aff9785b7e91a5
parent 499c3b12272b09bf3450107daeb25a8d0282affe
Author: Arthur Williams <taaparthur@disroot.org>
Date:   Sun,  6 Mar 2022 04:14:01 -0800

cp: don't abort when src and dest file are the same

The POSIX spec gives many options on how to handle this case, but it
also states that cp (and mv) should continue with remaining operands
regardless. We used to exit immediately, which violates the spec.

This change makes cp/mv not exit immediately in this case and
also won't cause the return value to be non-zero.

From `man 1p cp`:
   If source_file references the same file as dest_file, cp may write
   a diagnostic message to standard error; it shall do nothing more
   with source_file and shall go on to any remaining files.

Diffstat:
M
libutil/fnck.c
 | 
3
++-

1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libutil/fnck.c b/libutil/fnck.c
@@ -13,7 +13,8 @@ fnck(const char *a, const char *b,
 	    && !stat(b, &stb)
 	    && sta.st_dev == stb.st_dev
 	    && sta.st_ino == stb.st_ino) {
-		eprintf("%s -> %s: same file\n", a, b);
+		weprintf("%s -> %s: same file\n", a, b);
+		return;
 	}
 
 	if (fn(a, b, depth) < 0)