Got rid of some patch fuzz.
[rsync/rsync-patches.git] / remove-sent-files.diff
index 6a984c4..c507534 100644 (file)
@@ -4,9 +4,31 @@ command before "make":
     make proto
 
 
---- orig/io.c  2004-07-29 16:08:03
-+++ io.c       2004-07-03 20:17:10
-@@ -240,6 +240,14 @@ static void read_msg_fd(void)
+--- orig/generator.c   2005-02-16 15:35:52
++++ generator.c        2005-02-15 21:33:51
+@@ -38,6 +38,7 @@ extern int preserve_gid;
+ extern int preserve_times;
+ extern int omit_dir_times;
+ extern int delete_during;
++extern int remove_sent_files;
+ extern int update_only;
+ extern int opt_ignore_existing;
+ extern int inplace;
+@@ -510,6 +511,11 @@ static void recv_generator(char *fname, 
+                               rprintf(FINFO, "%s -> %s\n", safe_fname(fname),
+                                       safe_fname(file->u.link));
+                       }
++                      if (remove_sent_files && !dry_run) {
++                              char numbuf[4];
++                              SIVAL(numbuf, 0, ndx);
++                              io_multiplex_write(MSG_SUCCESS, numbuf, 4);
++                      }
+               }
+ #endif
+               return;
+--- orig/io.c  2005-02-03 02:04:20
++++ io.c       2005-02-15 21:33:51
+@@ -244,6 +244,14 @@ static void read_msg_fd(void)
                read_loop(fd, buf, 4);
                redo_list_add(IVAL(buf,0));
                break;
@@ -21,7 +43,7 @@ command before "make":
        case MSG_INFO:
        case MSG_ERROR:
        case MSG_LOG:
-@@ -673,6 +681,16 @@ static int readfd_unbuffered(int fd, cha
+@@ -677,6 +685,16 @@ static int readfd_unbuffered(int fd, cha
                        read_loop(fd, iobuf_in, remaining);
                        iobuf_in_ndx = 0;
                        break;
@@ -38,123 +60,143 @@ command before "make":
                case MSG_INFO:
                case MSG_ERROR:
                        if (remaining >= sizeof line) {
---- orig/main.c        2004-07-29 16:08:03
-+++ main.c     2004-07-22 03:06:20
-@@ -42,6 +42,7 @@ extern int list_only;
- extern int local_server;
+--- orig/main.c        2005-02-15 19:27:04
++++ main.c     2005-02-15 21:33:51
+@@ -33,12 +33,14 @@ extern int verbose;
+ extern int itemize_changes;
+ extern int blocking_io;
+ extern int delete_before;
++extern int remove_sent_files;
+ extern int daemon_over_rsh;
+ extern int do_stats;
+ extern int dry_run;
+ extern int list_only;
  extern int log_got_error;
  extern int module_id;
 +extern int need_messages_from_generator;
  extern int orig_umask;
+ extern int copy_links;
  extern int keep_dirlinks;
- extern int preserve_hard_links;
-@@ -612,6 +613,8 @@ void start_server(int f_in, int f_out, i
+@@ -442,6 +444,12 @@ static void do_server_sender(int f_in, i
+               exit_cleanup(RERR_SYNTAX);
+               return;
+       }
++      if (am_daemon && lp_read_only(module_id) && remove_sent_files) {
++              rprintf(FERROR,
++                  "ERROR: --remove-sent-files cannot be used with a read-only module\n");
++              exit_cleanup(RERR_SYNTAX);
++              return;
++      }
+       if (!relative_paths && !push_dir(dir)) {
+               rsyserr(FERROR, errno, "push_dir#3 %s failed",
+@@ -673,6 +681,8 @@ void start_server(int f_in, int f_out, i
  
        if (am_sender) {
                keep_dirlinks = 0; /* Must be disabled on the sender. */
 +              if (need_messages_from_generator)
-+                      io_start_multiplex_in(f_in);
++                      io_start_multiplex_in();
  
-               recv_exclude_list(f_in);
-               if (cvs_exclude)
-@@ -683,6 +686,9 @@ int client_run(int f_in, int f_out, pid_
+               recv_filter_list(f_in);
+               do_server_sender(f_in, f_out, argc, argv);
+@@ -750,6 +760,9 @@ int client_run(int f_in, int f_out, pid_
                exit_cleanup(status);
        }
  
 +      if (need_messages_from_generator && !read_batch)
-+              io_start_multiplex_out(f_out);
++              io_start_multiplex_out();
 +
        if (argc == 0)
-               list_only = 1;
+               list_only |= 1;
  
---- orig/options.c     2004-07-29 16:08:03
-+++ options.c  2004-07-16 20:09:54
-@@ -87,8 +87,10 @@ int size_only = 0;
- int bwlimit = 0;
- size_t bwlimit_writemax = 0;
+--- orig/options.c     2005-02-16 15:35:53
++++ options.c  2005-02-15 21:33:52
+@@ -59,6 +59,7 @@ int delete_during = 0;
+ int delete_before = 0;
  int delete_after = 0;
-+int delete_sent_files = 0;
+ int delete_excluded = 0;
++int remove_sent_files = 0;
+ int one_file_system = 0;
+ int protocol_version = PROTOCOL_VERSION;
+ int sparse_files = 0;
+@@ -93,6 +94,7 @@ int fuzzy_basis = 0;
+ size_t bwlimit_writemax = 0;
  int only_existing = 0;
  int opt_ignore_existing = 0;
 +int need_messages_from_generator = 0;
  int max_delete = 0;
+ OFF_T max_size = 0;
  int ignore_errors = 0;
- int modify_window = 0;
-@@ -266,6 +268,7 @@ void usage(enum logcode F)
+@@ -286,6 +288,7 @@ void usage(enum logcode F)
+   rprintf(F,"     --rsync-path=PATH       specify path to rsync on the remote machine\n");
+   rprintf(F,"     --existing              only update files that already exist on receiver\n");
+   rprintf(F,"     --ignore-existing       ignore files that already exist on receiving side\n");
++  rprintf(F,"     --remove-sent-files     sent files/symlinks are removed from sending side\n");
+   rprintf(F,"     --del                   an alias for --delete-during\n");
    rprintf(F,"     --delete                delete files that don't exist on the sending side\n");
-   rprintf(F,"     --delete-excluded       also delete excluded files on the receiving side\n");
-   rprintf(F,"     --delete-after          receiver deletes after transferring, not before\n");
-+  rprintf(F,"     --delete-sent-files     updated/sent files are removed from sending side\n");
-   rprintf(F,"     --ignore-errors         delete even if there are I/O errors\n");
-   rprintf(F,"     --max-delete=NUM        don't delete more than NUM files\n");
-   rprintf(F,"     --partial               keep partially transferred files\n");
-@@ -317,8 +320,8 @@ void usage(enum logcode F)
- }
- enum {OPT_VERSION = 1000, OPT_SENDER, OPT_EXCLUDE, OPT_EXCLUDE_FROM,
--      OPT_DELETE_AFTER, OPT_DELETE_EXCLUDED, OPT_LINK_DEST,
--      OPT_INCLUDE, OPT_INCLUDE_FROM, OPT_MODIFY_WINDOW,
-+      OPT_DELETE_AFTER, OPT_DELETE_EXCLUDED, OPT_DELETE_SENT_FILES,
-+      OPT_INCLUDE, OPT_INCLUDE_FROM, OPT_LINK_DEST, OPT_MODIFY_WINDOW,
-       OPT_READ_BATCH, OPT_WRITE_BATCH, OPT_TIMEOUT,
-       OPT_REFUSED_BASE = 9000};
-@@ -337,6 +340,7 @@ static struct poptOption long_options[] 
-   {"ignore-existing",  0,  POPT_ARG_NONE,   &opt_ignore_existing, 0, 0, 0 },
-   {"delete-after",     0,  POPT_ARG_NONE,   0,              OPT_DELETE_AFTER, 0, 0 },
-   {"delete-excluded",  0,  POPT_ARG_NONE,   0,              OPT_DELETE_EXCLUDED, 0, 0 },
-+  {"delete-sent-files",0,  POPT_ARG_NONE,   0,              OPT_DELETE_SENT_FILES, 0, 0 },
+   rprintf(F,"     --delete-before         receiver deletes before transfer (default)\n");
+@@ -369,6 +372,7 @@ static struct poptOption long_options[] 
+   {"delete-during",    0,  POPT_ARG_NONE,   &delete_during, 0, 0, 0 },
+   {"delete-after",     0,  POPT_ARG_NONE,   &delete_after, 0, 0, 0 },
+   {"delete-excluded",  0,  POPT_ARG_NONE,   &delete_excluded, 0, 0, 0 },
++  {"remove-sent-files",0,  POPT_ARG_NONE,   &remove_sent_files, 0, 0, 0 },
    {"force",            0,  POPT_ARG_NONE,   &force_delete, 0, 0, 0 },
    {"numeric-ids",      0,  POPT_ARG_NONE,   &numeric_ids, 0, 0, 0 },
-   {"exclude",          0,  POPT_ARG_STRING, 0,              OPT_EXCLUDE, 0, 0 },
-@@ -524,6 +528,11 @@ int parse_arguments(int *argc, const cha
-                       delete_mode = 1;
-                       break;
+   {"filter",          'f', POPT_ARG_STRING, 0, OPT_FILTER, 0, 0 },
+@@ -969,6 +973,17 @@ int parse_arguments(int *argc, const cha
+               return 0;
+       }
  
-+              case OPT_DELETE_SENT_FILES:
-+                      delete_sent_files = 1;
-+                      need_messages_from_generator = 1;
-+                      break;
++      if (remove_sent_files) {
++              /* We only want to infer this refusal of --remove-sent-files
++               * via the refusal of "delete", not any of the "delete-FOO"
++               * options. */
++              if (refused_delete && am_sender) {
++                      create_refuse_error(refused_delete);
++                      return 0;
++              }
++              need_messages_from_generator = 1;
++      }
 +
-               case OPT_EXCLUDE:
-                       if (am_server || sanitize_paths)
-                               return 0; /* Impossible... */
-@@ -1042,6 +1051,9 @@ void server_options(char **args,int *arg
-               }
-       }
+       *argv = poptGetArgs(pc);
+       *argc = count_args(*argv);
  
-+      if (delete_sent_files)
-+              args[ac++] = "--delete-sent-files";
+@@ -1411,6 +1426,9 @@ void server_options(char **args,int *arg
+       if (fuzzy_basis && am_sender)
+               args[ac++] = "--fuzzy";
++      if (remove_sent_files)
++              args[ac++] = "--remove-sent-files";
 +
        *argc = ac;
        return;
  
---- orig/receiver.c    2004-07-29 16:08:03
-+++ receiver.c 2004-07-29 16:11:11
-@@ -46,6 +46,7 @@ extern char *backup_dir;
- extern char *backup_suffix;
- extern int backup_suffix_len;
+--- orig/receiver.c    2005-02-16 15:35:53
++++ receiver.c 2005-02-15 21:33:52
+@@ -43,6 +43,7 @@ extern int basis_dir_cnt;
+ extern int make_backups;
+ extern int do_progress;
  extern int cleanup_got_literal;
-+extern int delete_sent_files;
++extern int remove_sent_files;
  extern int module_id;
  extern int ignore_errors;
  extern int orig_umask;
-@@ -341,7 +342,7 @@ int recv_files(int f_in, struct file_lis
+@@ -310,7 +311,7 @@ int recv_files(int f_in, struct file_lis
        char *fname, fbuf[MAXPATHLEN];
        char template[MAXPATHLEN];
        char fnametmp[MAXPATHLEN];
 -      char *fnamecmp, *partialptr;
 +      char *fnamecmp, *partialptr, numbuf[4];
        char fnamecmpbuf[MAXPATHLEN];
+       uchar *delayed_bits = NULL;
        struct file_struct *file;
-       struct stats initial_stats;
-@@ -576,7 +577,12 @@ int recv_files(int f_in, struct file_lis
+@@ -588,7 +589,12 @@ int recv_files(int f_in, struct file_lis
  
                cleanup_disable();
  
 -              if (!recv_ok) {
 +              if (recv_ok) {
-+                      if (delete_sent_files) {
++                      if (remove_sent_files) {
 +                              SIVAL(numbuf, 0, i);
 +                              send_msg(MSG_SUCCESS, numbuf, 4);
 +                      }
@@ -162,7 +204,7 @@ command before "make":
                        int msgtype = csum_length == SUM_LENGTH || read_batch ?
                                FERROR : FINFO;
                        if (msgtype == FERROR || verbose) {
-@@ -600,9 +606,8 @@ int recv_files(int f_in, struct file_lis
+@@ -612,9 +618,8 @@ int recv_files(int f_in, struct file_lis
                                        keptstr, redostr);
                        }
                        if (csum_length != SUM_LENGTH) {
@@ -174,57 +216,52 @@ command before "make":
                        }
                }
        }
---- orig/rsync.h       2004-07-29 16:08:03
-+++ rsync.h    2004-07-03 20:17:10
-@@ -60,6 +60,7 @@
- #define FLAG_TOP_DIR (1<<0)
- #define FLAG_HLINK_EOL (1<<1) /* generator only */
- #define FLAG_MOUNT_POINT (1<<2)       /* sender only */
-+#define FLAG_SENT (1<<7)      /* sender only */
- /* update this if you make incompatible changes */
- #define PROTOCOL_VERSION 28
-@@ -126,6 +127,7 @@ enum logcode { FERROR=1, FINFO=2, FLOG=3
- /* Messages types that are sent over the message channel.  The logcode
-  * values must all be present here with identical numbers. */
- enum msgcode {
-+      MSG_SUCCESS=6,  /* successfully updated indicated flist index */
-       MSG_DONE=5,     /* current phase is done */
-       MSG_REDO=4,     /* reprocess indicated flist index */
-       MSG_ERROR=FERROR, MSG_INFO=FINFO, MSG_LOG=FLOG, /* remote logging */
---- orig/rsync.yo      2004-07-29 16:08:04
-+++ rsync.yo   2004-07-03 20:17:10
-@@ -314,6 +314,7 @@ verb(
+--- orig/rsync.yo      2005-02-16 15:35:54
++++ rsync.yo   2005-02-15 21:33:55
+@@ -332,6 +332,7 @@ to the detailed description below for a 
+      --rsync-path=PATH       specify path to rsync on the remote machine
+      --existing              only update files that already exist
+      --ignore-existing       ignore files that already exist on receiver
++     --remove-sent-files     sent files/symlinks are removed from sender
+      --del                   an alias for --delete-during
       --delete                delete files that don't exist on sender
-      --delete-excluded       also delete excluded files on receiver
-      --delete-after          receiver deletes after transfer, not before
-+     --delete-sent-files     updated/sent files are removed from sender
-      --ignore-errors         delete even if there are I/O errors
-      --max-delete=NUM        don't delete more than NUM files
-      --partial               keep partially transferred files
-@@ -619,6 +620,11 @@ receiving side before transferring files
- sufficient space on the receiving filesystem. If you want to delete
- after transferring, use the --delete-after switch. Implies --delete.
+      --delete-before         receiver deletes before transfer (default)
+@@ -665,6 +666,11 @@ dit(bf(--ignore-existing))
+ This tells rsync not to update files that already exist on
+ the destination.
  
-+dit(bf(--delete-sent-files)) This tells rsync to remove the source files
-+on the sending side that are successfully transferred to the receiving
-+side.  Directories are not removed, nor are files that are identical on
-+both systems.
++dit(bf(--remove-sent-files)) This tells rsync to remove from the sending
++side the files and/or symlinks that are newly created or whose content is
++updated on the receiving side.  Directories and devices are not removed,
++nor are files/symlinks whose attributes are merely changed.
 +
- dit(bf(--ignore-errors)) Tells --delete to go ahead and delete files
- even when there are I/O errors.
+ dit(bf(--delete)) This tells rsync to delete extraneous files from the
+ receiving side (ones that aren't on the sending side), but only for the
+ directories that are being synchronized.  You must have asked rsync to
+--- orig/rsyncd.conf.yo        2005-02-16 15:35:54
++++ rsyncd.conf.yo     2005-02-15 21:33:55
+@@ -443,6 +443,10 @@ quote(tt(    refuse options = c delete))
+ The reason the above refuses all delete options is that the options imply
+ bf(--delete), and implied options are refused just like explicit options.
++As an additional safety feature, the refusal of "delete" also refuses
++bf(remove-sent-files) when the daemon is the sender; if you want the latter
++without the former, instead refuse "delete-*" -- that refuses all the
++delete modes without affecting bf(--remove-sent-files).
  
---- orig/sender.c      2004-07-26 16:27:00
-+++ sender.c   2004-07-26 16:49:19
-@@ -26,6 +26,7 @@ extern int io_error;
+ When an option is refused, the server prints an error message and exits.
+ To prevent all compression, you can use "dont compress = *" (see below)
+--- orig/sender.c      2005-02-16 15:35:54
++++ sender.c   2005-02-15 21:33:55
+@@ -28,6 +28,7 @@ extern int io_error;
  extern int dry_run;
  extern int am_server;
  extern int am_daemon;
-+extern int delete_sent_files;
++extern int remove_sent_files;
  extern int protocol_version;
- extern struct stats stats;
-@@ -107,7 +108,29 @@ static struct sum_struct *receive_sums(i
+ extern int updating_basis_file;
+ extern int make_backups;
+@@ -98,7 +99,32 @@ static struct sum_struct *receive_sums(i
        return s;
  }
  
@@ -236,25 +273,28 @@ command before "make":
 +      struct file_struct *file;
 +      unsigned int offset;
 +
-+      if (!the_flist)
++      if (!the_flist || i < 0 || i >= the_flist->count)
 +              return;
 +
 +      file = the_flist->files[i];
-+      if (!(file->flags & FLAG_SENT))
-+              return; /* We didn't send it -- impossible! */
-+      if (file->basedir) {
++      /* The generator might tell us about symlinks we didn't send. */
++      if (!(file->flags & FLAG_SENT) && !S_ISLNK(file->mode))
++              return;
++      if (file->dir.root) {
 +              offset = stringjoin(fname, sizeof fname,
-+                                  file->basedir, "/", NULL);
++                                  file->dir.root, "/", NULL);
 +      } else
 +              offset = 0;
 +      f_name_to(file, fname + offset);
-+      if (delete_sent_files && do_unlink(fname) == 0 && verbose > 0)
-+              rprintf(FINFO, "sender removed %s\n", fname + offset);
++      if (remove_sent_files && do_unlink(fname) == 0 && verbose) {
++              rprintf(FINFO, "sender removed %s\n",
++                      safe_fname(fname + offset));
++      }
 +}
  
  void send_files(struct file_list *flist, int f_out, int f_in)
  {
-@@ -125,6 +148,8 @@ void send_files(struct file_list *flist,
+@@ -117,6 +143,8 @@ void send_files(struct file_list *flist,
        if (verbose > 2)
                rprintf(FINFO, "send_files starting\n");
  
@@ -263,7 +303,7 @@ command before "make":
        while (1) {
                unsigned int offset;
  
-@@ -248,6 +273,9 @@ void send_files(struct file_list *flist,
+@@ -281,6 +309,9 @@ void send_files(struct file_list *flist,
                        rprintf(FINFO, "sender finished %s\n",
                                safe_fname(fname));
                }
@@ -271,5 +311,5 @@ command before "make":
 +              /* Flag that we actually sent this entry. */
 +              file->flags |= FLAG_SENT;
        }
+       make_backups = save_make_backups;
  
-       if (verbose > 2)