- Changed the calling args to delete_file() to take an st_mode instead

[rsync/rsync.git] / generator.c

diff --git a/generator.c b/generator.c
index 188f51f..e3ebd45 100644 (file)
--- a/generator.c
+++ b/generator.c
@@ -25,6 +25,7 @@
 
 extern int verbose;
 extern int dry_run;
+extern int itemize_changes;
 extern int relative_paths;
 extern int keep_dirlinks;
 extern int preserve_links;
@@ -92,28 +93,29 @@ static void itemize(struct file_struct *file, int statret, STRUCT_STAT *st,
        if (statret >= 0) {
                if (S_ISREG(file->mode) && file->length != st->st_size)
                        iflags |= ITEM_REPORT_SIZE;
+               if (!(sflags & SID_NO_DEST_AND_NO_UPDATE)) {
+                       int keep_time = !preserve_times ? 0
+                           : S_ISDIR(file->mode) ? !omit_dir_times
+                           : !S_ISLNK(file->mode);
+
+                       if ((iflags & ITEM_UPDATING && !keep_time)
+                           || (keep_time && file->modtime != st->st_mtime))
+                               iflags |= ITEM_REPORT_TIME;
+                       if (preserve_perms && file->mode != st->st_mode)
+                               iflags |= ITEM_REPORT_PERMS;
+                       if (preserve_uid && am_root && file->uid != st->st_uid)
+                               iflags |= ITEM_REPORT_OWNER;
+                       if (preserve_gid && file->gid != GID_NONE
+                           && st->st_gid != file->gid)
+                               iflags |= ITEM_REPORT_GROUP;
+               }
        } else
-               iflags |= ITEM_IS_NEW;
-       if (statret >= 0 && !(sflags & SID_NO_DEST_AND_NO_UPDATE)) {
-               int keep_time = !preserve_times ? 0
-                   : S_ISDIR(file->mode) ? !omit_dir_times : !S_ISLNK(file->mode);
-
-               if ((iflags & ITEM_UPDATING && !keep_time)
-                   || (keep_time && file->modtime != st->st_mtime))
-                       iflags |= ITEM_REPORT_TIME;
-               if (preserve_perms && file->mode != st->st_mode)
-                       iflags |= ITEM_REPORT_PERMS;
-               if (preserve_uid && am_root && file->uid != st->st_uid)
-                       iflags |= ITEM_REPORT_OWNER;
-               if (preserve_gid && file->gid != GID_NONE && st->st_gid != file->gid)
-                       iflags |= ITEM_REPORT_GROUP;
-       }
-
-       if (iflags && !read_batch) {
+               iflags |= ITEM_IS_NEW | ITEM_UPDATING;
+
+       if ((iflags || verbose > 1) && !read_batch) {
                if (ndx >= 0)
                        write_int(f_out, ndx);
-               write_byte(f_out, iflags);
-               write_byte(f_out, iflags >> 8);
+               write_shortint(f_out, iflags);
        }
 }
 
@@ -345,6 +347,8 @@ static void recv_generator(char *fname, struct file_list *flist,
        char *fnamecmp, *partialptr, *backupptr = NULL;
        char fnamecmpbuf[MAXPATHLEN];
        uchar fnamecmp_type;
+       int maybe_DEL_TERSE = itemize_changes ? 0 : DEL_TERSE;
+       int maybe_PERMS_REPORT = itemize_changes ? 0 : PERMS_REPORT;
 
        if (list_only)
                return;
@@ -427,7 +431,7 @@ static void recv_generator(char *fname, struct file_list *flist,
                 * we need to delete it.  If it doesn't exist, then
                 * (perhaps recursively) create it. */
                if (statret == 0 && !S_ISDIR(st.st_mode)) {
-                       delete_file(fname, DEL_TERSE);
+                       delete_file(fname, st.st_mode, maybe_DEL_TERSE);
                        statret = -1;
                }
                if (dry_run && statret != 0 && missing_below < 0) {
@@ -474,11 +478,10 @@ static void recv_generator(char *fname, struct file_list *flist,
                        return;
                }
                if (statret == 0) {
-                       int dflag = S_ISDIR(st.st_mode) ? DEL_DIR : 0;
                        char lnk[MAXPATHLEN];
                        int len;
 
-                       if (!dflag
+                       if (!S_ISDIR(st.st_mode)
                            && (len = readlink(fname, lnk, MAXPATHLEN-1)) > 0) {
                                lnk[len] = 0;
                                /* A link already pointing to the
@@ -490,13 +493,18 @@ static void recv_generator(char *fname, struct file_list *flist,
                                                        f_out, ndx);
                                        }
                                        set_perms(fname, file, &st,
-                                                 PERMS_REPORT);
+                                                 maybe_PERMS_REPORT);
                                        return;
                                }
                        }
                        /* Not the right symlink (or not a symlink), so
                         * delete it. */
-                       delete_file(fname, dflag | DEL_TERSE);
+                       if (S_ISLNK(st.st_mode))
+                               delete_file(fname, st.st_mode, DEL_TERSE);
+                       else {
+                               delete_file(fname, st.st_mode, maybe_DEL_TERSE);
+                               statret = -1;
+                       }
                }
                if (do_symlink(file->u.link,fname) != 0) {
                        rsyserr(FERROR, errno, "symlink %s -> \"%s\" failed",
@@ -519,12 +527,12 @@ static void recv_generator(char *fname, struct file_list *flist,
                if (statret != 0 ||
                    st.st_mode != file->mode ||
                    st.st_rdev != file->u.rdev) {
-                       int dflag = S_ISDIR(st.st_mode) ? DEL_DIR : 0;
-                       if (protocol_version >= 29) {
-                               itemize(file, statret, &st, SID_UPDATING,
-                                       f_out, ndx);
+                       if (IS_DEVICE(st.st_mode))
+                               delete_file(fname, st.st_mode, DEL_TERSE);
+                       else {
+                               delete_file(fname, st.st_mode, maybe_DEL_TERSE);
+                               statret = -1;
                        }
-                       delete_file(fname, dflag | DEL_TERSE);
                        if (verbose > 2) {
                                rprintf(FINFO,"mknod(%s,0%o,0x%x)\n",
                                        safe_fname(fname),
@@ -535,7 +543,10 @@ static void recv_generator(char *fname, struct file_list *flist,
                                        full_fname(fname));
                        } else {
                                set_perms(fname,file,NULL,0);
-                               if (verbose && protocol_version < 29) {
+                               if (protocol_version >= 29) {
+                                       itemize(file, statret, &st, SID_UPDATING,
+                                               f_out, ndx);
+                               } else if (verbose) {
                                        rprintf(FINFO, "%s\n",
                                                safe_fname(fname));
                                }
@@ -545,7 +556,7 @@ static void recv_generator(char *fname, struct file_list *flist,
                                itemize(file, statret, &st, 0,
                                        f_out, ndx);
                        }
-                       set_perms(fname, file, &st, PERMS_REPORT);
+                       set_perms(fname, file, &st, maybe_PERMS_REPORT);
                }
                return;
        }
@@ -618,8 +629,7 @@ static void recv_generator(char *fname, struct file_list *flist,
        }
 
        if (statret == 0 && !S_ISREG(st.st_mode)) {
-               int dflag = S_ISDIR(st.st_mode) ? DEL_DIR : 0;
-               if (delete_file(fname, dflag | DEL_TERSE) != 0)
+               if (delete_file(fname, st.st_mode, maybe_DEL_TERSE) != 0)
                        return;
                statret = -1;
                stat_errno = ENOENT;
@@ -689,7 +699,7 @@ static void recv_generator(char *fname, struct file_list *flist,
                                f_out, ndx);
                }
                if (fnamecmp_type == FNAMECMP_FNAME)
-                       set_perms(fname, file, &st, PERMS_REPORT);
+                       set_perms(fname, file, &st, maybe_PERMS_REPORT);
                return;
        }