Got rid of some patch fuzz.
[rsync/rsync-patches.git] / remove-sent-files.diff
index 4c48179..c507534 100644 (file)
@@ -4,8 +4,30 @@ command before "make":
     make proto
 
 
---- orig/io.c  2005-01-19 20:11:10
-+++ io.c       2005-01-10 10:49:17
+--- 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));
@@ -38,26 +60,46 @@ command before "make":
                case MSG_INFO:
                case MSG_ERROR:
                        if (remaining >= sizeof line) {
---- orig/main.c        2005-01-17 23:11:45
-+++ main.c     2004-08-13 08:24:23
-@@ -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;
-@@ -648,6 +649,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();
  
-               recv_exclude_list(f_in);
-               if (cvs_exclude)
-@@ -730,6 +733,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);
        }
  
@@ -65,83 +107,96 @@ command before "make":
 +              io_start_multiplex_out();
 +
        if (argc == 0)
-               list_only = 1;
+               list_only |= 1;
  
---- orig/options.c     2005-01-20 23:05:34
-+++ options.c  2005-01-10 10:46:47
-@@ -89,8 +89,10 @@ int daemon_bwlimit = 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;
-@@ -274,6 +276,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,"     --max-size=SIZE         don't transfer any file larger than SIZE\n");
-@@ -342,6 +345,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,   &delete_sent_files, 0, 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 },
-@@ -858,6 +862,9 @@ int parse_arguments(int *argc, const cha
-       if (relative_paths < 0)
-               relative_paths = files_from? 1 : 0;
+   {"filter",          'f', POPT_ARG_STRING, 0, OPT_FILTER, 0, 0 },
+@@ -969,6 +973,17 @@ int parse_arguments(int *argc, const cha
+               return 0;
+       }
  
-+      if (delete_sent_files)
++      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;
++      }
 +
        *argv = poptGetArgs(pc);
        *argc = count_args(*argv);
  
-@@ -1241,6 +1248,9 @@ void server_options(char **args,int *arg
-       if (!implied_dirs && !am_sender)
-               args[ac++] = "--no-implied-dirs";
+@@ -1411,6 +1426,9 @@ void server_options(char **args,int *arg
+       if (fuzzy_basis && am_sender)
+               args[ac++] = "--fuzzy";
  
-+      if (delete_sent_files)
-+              args[ac++] = "--delete-sent-files";
++      if (remove_sent_files)
++              args[ac++] = "--remove-sent-files";
 +
        *argc = ac;
        return;
  
---- orig/receiver.c    2005-01-21 00:35:26
-+++ receiver.c 2004-08-13 08:38:51
-@@ -47,6 +47,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;
-@@ -322,7 +323,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;
-@@ -568,7 +569,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);
 +                      }
@@ -149,7 +204,7 @@ command before "make":
                        int msgtype = csum_length == SUM_LENGTH || read_batch ?
                                FERROR : FINFO;
                        if (msgtype == FERROR || verbose) {
-@@ -592,9 +598,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) {
@@ -161,57 +216,52 @@ command before "make":
                        }
                }
        }
---- orig/rsync.h       2005-01-21 00:35:26
-+++ rsync.h    2004-07-03 20:17:10
-@@ -60,6 +60,7 @@
- #define FLAG_DEL_START (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 29
-@@ -142,6 +143,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      2005-01-20 19:47:08
-+++ rsync.yo   2004-07-03 20:17:10
-@@ -343,6 +343,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
-      --max-size=SIZE         don't transfer any file larger than SIZE
-@@ -695,6 +696,11 @@ One reason to use --delete-after is to a
- the transfer (while the receiving side is scanned for deletions) as this
- delay might cause the transfer to timeout.  
+      --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      2005-01-17 23:11:46
-+++ 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 int updating_basis_file;
  extern int make_backups;
-@@ -94,7 +95,29 @@ static struct sum_struct *receive_sums(i
+@@ -98,7 +99,32 @@ static struct sum_struct *receive_sums(i
        return s;
  }
  
@@ -227,21 +277,24 @@ command before "make":
 +              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)
  {
-@@ -113,6 +136,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");
  
@@ -250,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));
                }