Got rid of some patch fuzz.
[rsync/rsync-patches.git] / remove-sent-files.diff
index 9da8fc3..c507534 100644 (file)
@@ -4,8 +4,30 @@ command before "make":
     make proto
 
 
+--- 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-01-10 10:49:17
++++ 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,13 +60,13 @@ command before "make":
                case MSG_INFO:
                case MSG_ERROR:
                        if (remaining >= sizeof line) {
---- orig/main.c        2005-02-07 20:41:56
-+++ main.c     2005-02-05 01:24:16
-@@ -32,12 +32,14 @@ extern int am_daemon;
- extern int verbose;
+--- 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 delete_sent_files;
++extern int remove_sent_files;
  extern int daemon_over_rsh;
  extern int do_stats;
  extern int dry_run;
@@ -55,20 +77,20 @@ command before "make":
  extern int orig_umask;
  extern int copy_links;
  extern int keep_dirlinks;
-@@ -440,6 +442,12 @@ static void do_server_sender(int f_in, 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) && delete_sent_files) {
++      if (am_daemon && lp_read_only(module_id) && remove_sent_files) {
 +              rprintf(FERROR,
-+                  "ERROR: --delete-sent-files cannot be used with a read-only module\n");
++                  "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",
-@@ -670,6 +678,8 @@ void start_server(int f_in, int f_out, i
+@@ -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. */
@@ -77,7 +99,7 @@ command before "make":
  
                recv_filter_list(f_in);
                do_server_sender(f_in, f_out, argc, argv);
-@@ -747,6 +757,9 @@ int client_run(int f_in, int f_out, pid_
+@@ -750,6 +760,9 @@ int client_run(int f_in, int f_out, pid_
                exit_cleanup(status);
        }
  
@@ -87,17 +109,17 @@ command before "make":
        if (argc == 0)
                list_only |= 1;
  
---- orig/options.c     2005-02-11 18:21:45
-+++ options.c  2005-01-23 07:30:51
+--- 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_excluded = 0;
-+int delete_sent_files = 0;
++int remove_sent_files = 0;
  int one_file_system = 0;
  int protocol_version = PROTOCOL_VERSION;
  int sparse_files = 0;
-@@ -92,6 +93,7 @@ int bwlimit = 0;
+@@ -93,6 +94,7 @@ int fuzzy_basis = 0;
  size_t bwlimit_writemax = 0;
  int only_existing = 0;
  int opt_ignore_existing = 0;
@@ -105,53 +127,61 @@ command before "make":
  int max_delete = 0;
  OFF_T max_size = 0;
  int ignore_errors = 0;
-@@ -289,6 +291,7 @@ void usage(enum logcode F)
-   rprintf(F,"     --delete-during         receiver deletes during transfer, not before\n");
-   rprintf(F,"     --delete-after          receiver deletes after transfer, not before\n");
-   rprintf(F,"     --delete-excluded       also delete excluded files on the receiving side\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,"     --force                 force deletion of directories even if not empty\n");
-   rprintf(F,"     --max-delete=NUM        don't delete more than NUM files\n");
-@@ -364,6 +367,7 @@ static struct poptOption long_options[] 
+@@ -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-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 },
-+  {"delete-sent-files",0,  POPT_ARG_NONE,   &delete_sent_files, 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 },
    {"filter",          'f', POPT_ARG_STRING, 0, OPT_FILTER, 0, 0 },
-@@ -955,6 +959,9 @@ int parse_arguments(int *argc, const cha
+@@ -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);
  
-@@ -1382,6 +1389,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-02-11 10:53:14
-+++ receiver.c 2004-08-13 08:38:51
-@@ -41,6 +41,7 @@ extern int basis_dir_cnt;
+--- 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;
-@@ -276,7 +277,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];
@@ -160,13 +190,13 @@ command before "make":
        char fnamecmpbuf[MAXPATHLEN];
        uchar *delayed_bits = NULL;
        struct file_struct *file;
-@@ -533,7 +534,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);
 +                      }
@@ -174,7 +204,7 @@ command before "make":
                        int msgtype = csum_length == SUM_LENGTH || read_batch ?
                                FERROR : FINFO;
                        if (msgtype == FERROR || verbose) {
-@@ -557,9 +563,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) {
@@ -186,57 +216,52 @@ command before "make":
                        }
                }
        }
---- orig/rsync.h       2005-02-07 20:41:57
-+++ rsync.h    2005-02-03 02:04:59
-@@ -61,6 +61,7 @@
- #define FLAG_HLINK_EOL (1<<1) /* generator only */
- #define FLAG_MOUNT_POINT (1<<2)       /* sender only */
- #define FLAG_DEL_HERE (1<<3)  /* receiver/generator */
-+#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-02-11 23:14:49
-+++ rsync.yo   2005-02-03 02:05:29
-@@ -338,6 +338,7 @@ to the detailed description below for a 
-      --delete-during         receiver deletes during xfer, not before
-      --delete-after          receiver deletes after transfer, not before
-      --delete-excluded       also delete excluded files on receiver
-+     --delete-sent-files     updated/sent files are removed from sender
-      --ignore-errors         delete even if there are I/O errors
-      --force                 force deletion of dirs even if not empty
-      --max-delete=NUM        don't delete more than NUM files
-@@ -728,6 +729,11 @@ receiving side that are not on the sendi
- delete any files on the receiving side that are excluded (see bf(--exclude)).
- See bf(--delete) (which is implied) for more details on file-deletion.
+--- 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-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 bf(--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-02-11 22:24:19
-+++ 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;
  }
  
@@ -252,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");
  
@@ -275,7 +303,7 @@ command before "make":
        while (1) {
                unsigned int offset;
  
-@@ -254,6 +279,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));
                }