Partial merge with recent revisions from FreeBSD:
authorChris Pressey <cpressey@dragonflybsd.org>
Mon, 14 Feb 2005 02:09:12 +0000 (02:09 +0000)
committerChris Pressey <cpressey@dragonflybsd.org>
Mon, 14 Feb 2005 02:09:12 +0000 (02:09 +0000)
- pass -1 to setfile() (cp.c:1.49)
- When -R is not specified, fail to copy the contents of dangling
  symlinks instead of making a copy of the link itself, as required
  by SUSv3 (cp.c:1.36)
- Fix the case of a trailing slash following the destination
  mistakenly creating a file instead of a directory (cp.c:1.34)
- Mechanically kill hard sentence breaks in manual page (cp.1:1.29)
- Clarify what -p option does and spell "set-{user,group}-ID" with
  hyphens (cp.1:1.27)

Submitted-by: Sarunas Vancevicius <svan_at_redbrick_dot_dcu_dot_ie>
bin/cp/cp.1
bin/cp/cp.c

index 6efb3e8..c4c35f5 100644 (file)
@@ -34,7 +34,7 @@
 .\"
 .\"    @(#)cp.1        8.3 (Berkeley) 4/18/94
 .\" $FreeBSD: src/bin/cp/cp.1,v 1.16.2.6 2002/08/10 13:20:19 johan Exp $
-.\" $DragonFly: src/bin/cp/cp.1,v 1.2 2003/06/17 04:22:49 dillon Exp $
+.\" $DragonFly: src/bin/cp/cp.1,v 1.3 2005/02/14 02:09:12 cpressey Exp $
 .\"
 .Dd August 10, 2002
 .Dt CP 1
@@ -154,21 +154,22 @@ options.)
 .It Fl p
 Cause
 .Nm
-to preserve in the copy as many of the modification time, access time,
-file flags, file mode, user ID, and group ID as allowed by permissions.
+to preserve the following attributes of each source
+file in the copy: modification time, access time,
+file flags, file mode, user ID, and group ID, as allowed by permissions.
 .Pp
 If the user ID and group ID cannot be preserved, no error message
 is displayed and the exit value is not altered.
 .Pp
-If the source file has its set user ID bit on and the user ID cannot
-be preserved, the set user ID bit is not preserved
+If the source file has its set-user-ID bit on and the user ID cannot
+be preserved, the set-user-ID bit is not preserved
 in the copy's permissions.
-If the source file has its set group ID bit on and the group ID cannot
-be preserved, the set group ID bit is not preserved
+If the source file has its set-group-ID bit on and the group ID cannot
+be preserved, the set-group-ID bit is not preserved
 in the copy's permissions.
-If the source file has both its set user ID and set group ID bits on,
+If the source file has both its set-user-ID and set-group-ID bits on,
 and either the user ID or group ID cannot be preserved, neither
-the set user ID nor set group ID bits are preserved in the copy's
+the set-user-ID nor set-group-ID bits are preserved in the copy's
 permissions.
 .It Fl v
 Cause
@@ -177,7 +178,8 @@ to be verbose, showing files as they are copied.
 .El
 .Pp
 For each destination file that already exists, its contents are
-overwritten if permissions allow.  Its mode, user ID, and group
+overwritten if permissions allow.
+Its mode, user ID, and group
 ID are unchanged unless the
 .Fl p
 option was specified.
@@ -195,13 +197,13 @@ used as modified by the file mode creation mask
 .Pf ( Ic umask ,
 see
 .Xr csh 1 ) .
-If the source file has its set user ID bit on, that bit is removed
+If the source file has its set-user-ID bit on, that bit is removed
 unless both the source file and the destination file are owned by the
 same user.
-If the source file has its set group ID bit on, that bit is removed
+If the source file has its set-group-ID bit on, that bit is removed
 unless both the source file and the destination file are in the same
 group and the user is a member of that group.
-If both the set user ID and set group ID bits are set, all of the above
+If both the set-user-ID and set-group-ID bits are set, all of the above
 conditions must be fulfilled or both bits are removed.
 .Pp
 Appropriate permissions are required for file creation or overwriting.
index 2ce9653..bae164f 100644 (file)
@@ -36,7 +36,7 @@
  * @(#) Copyright (c) 1988, 1993, 1994 The Regents of the University of California.  All rights reserved.
  * @(#)cp.c    8.2 (Berkeley) 4/1/94
  * $FreeBSD: src/bin/cp/cp.c,v 1.24.2.7 2002/09/24 12:41:04 mckay Exp $
- * $DragonFly: src/bin/cp/cp.c,v 1.9 2004/11/07 20:54:51 eirikn Exp $
+ * $DragonFly: src/bin/cp/cp.c,v 1.10 2005/02/14 02:09:12 cpressey Exp $
  */
 
 /*
@@ -88,7 +88,7 @@ main(int argc, char **argv)
 {
        struct stat to_stat, tmp_stat;
        enum op type;
-       int Hflag, Lflag, Pflag, ch, fts_options, r;
+       int Hflag, Lflag, Pflag, ch, has_trailing_slash, r, fts_options;
        char *target;
 
        Hflag = Lflag = Pflag = 0;
@@ -160,7 +160,7 @@ main(int argc, char **argv)
                }
        } else {
                fts_options &= ~FTS_PHYSICAL;
-               fts_options |= FTS_LOGICAL;
+               fts_options |= FTS_LOGICAL | FTS_COMFOLLOW;
        }
 
        /* Save the target base in "to". */
@@ -172,7 +172,9 @@ main(int argc, char **argv)
                *to.p_end++ = '.';
                *to.p_end = 0;
        }
-        STRIP_TRAILING_SLASH(to);
+       has_trailing_slash = (to.p_end[-1] == '/');
+       if (has_trailing_slash)
+               STRIP_TRAILING_SLASH(to);
        to.target_end = to.p_end;
 
        /* Set end of argument list for fts(3). */
@@ -222,6 +224,13 @@ main(int argc, char **argv)
                                type = FILE_TO_FILE;
                } else
                        type = FILE_TO_FILE;
+               
+               if (has_trailing_slash && type == FILE_TO_FILE) {
+                       if (r == -1)
+                               errx(1, "directory %s does not exist", to.p_path);
+                       else
+                               errx(1, "%s is not a directory", to.p_path);
+               }
        } else
                /*
                 * Case (2).  Target is a directory.
@@ -339,7 +348,7 @@ copy(char **argv, enum op type, int fts_options)
                         * normally want to preserve them on directories.
                         */
                        if (pflag) {
-                               if (setfile(curr->fts_statp, 0))
+                               if (setfile(curr->fts_statp, -1))
                                    rval = 1;
                        } else {
                                mode = curr->fts_statp->st_mode;
@@ -378,8 +387,16 @@ copy(char **argv, enum op type, int fts_options)
 
                switch (curr->fts_statp->st_mode & S_IFMT) {
                case S_IFLNK:
-                       if (copy_link(curr, !dne))
-                               badcp = rval = 1;
+                       /* Catch special case of a non-dangling symlink */
+                       if ((fts_options & FTS_LOGICAL) ||
+                           ((fts_options & FTS_COMFOLLOW) &&
+                           curr->fts_level == 0)) {
+                               if (copy_file(curr, dne))
+                                       badcp = rval = 1;
+                       } else {        
+                               if (copy_link(curr, !dne))
+                                       badcp = rval = 1;
+                       }
                        break;
                case S_IFDIR:
                        if (!Rflag && !rflag) {