Need to make sure that the destination file doesn't exist before we
[rsync/rsync-patches.git] / remove-sent-files.diff
index 65e6a7b..baf5df6 100644 (file)
@@ -4,9 +4,9 @@ command before "make":
     make proto
 
 
---- io.c       27 May 2004 22:09:31 -0000      1.122
-+++ io.c       4 Jun 2004 05:21:59 -0000
-@@ -223,6 +223,14 @@ static void read_msg_fd(void)
+--- orig/io.c  2004-08-02 02:44:26
++++ io.c       2004-07-03 20:17:10
+@@ -240,6 +240,14 @@ static void read_msg_fd(void)
                read_loop(fd, buf, 4);
                redo_list_add(IVAL(buf,0));
                break;
@@ -21,9 +21,9 @@ command before "make":
        case MSG_INFO:
        case MSG_ERROR:
        case MSG_LOG:
-@@ -637,6 +645,16 @@ static int read_unbuffered(int fd, char 
-                       read_loop(fd, buffer, remaining);
-                       bufferIdx = 0;
+@@ -673,6 +681,16 @@ static int readfd_unbuffered(int fd, cha
+                       read_loop(fd, iobuf_in, remaining);
+                       iobuf_in_ndx = 0;
                        break;
 +              case MSG_SUCCESS:
 +                      if (remaining != 4) {
@@ -38,54 +38,49 @@ command before "make":
                case MSG_INFO:
                case MSG_ERROR:
                        if (remaining >= sizeof line) {
---- main.c     19 May 2004 22:19:19 -0000      1.195
-+++ main.c     4 Jun 2004 05:22:00 -0000
+--- orig/main.c        2004-09-18 01:49:33
++++ main.c     2004-08-13 08:24:23
 @@ -42,6 +42,7 @@ extern int list_only;
  extern int local_server;
  extern int log_got_error;
  extern int module_id;
 +extern int need_messages_from_generator;
  extern int orig_umask;
- extern int preserve_hard_links;
- extern int protocol_version;
-@@ -572,6 +573,8 @@ void start_server(int f_in, int f_out, i
-               io_start_multiplex_out(f_out);
+ extern int copy_links;
+ extern int keep_dirlinks;
+@@ -621,6 +622,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);
-               if (!read_batch) {
-                       recv_exclude_list(f_in);
-                       if (cvs_exclude)
-@@ -638,6 +641,9 @@ int client_run(int f_in, int f_out, pid_
++                      io_start_multiplex_in();
+               recv_exclude_list(f_in);
+               if (cvs_exclude)
+@@ -692,6 +695,9 @@ int client_run(int f_in, int f_out, pid_
                exit_cleanup(status);
        }
  
-+      if (need_messages_from_generator)
-+              io_start_multiplex_out(f_out);
++      if (need_messages_from_generator && !read_batch)
++              io_start_multiplex_out();
 +
        if (argc == 0)
                list_only = 1;
  
---- options.c  27 May 2004 21:51:53 -0000      1.153
-+++ options.c  4 Jun 2004 05:22:00 -0000
-@@ -85,6 +85,7 @@ int size_only = 0;
+--- orig/options.c     2004-09-20 05:10:48
++++ options.c  2004-07-16 20:09:54
+@@ -87,8 +87,10 @@ int size_only = 0;
  int bwlimit = 0;
  size_t bwlimit_writemax = 0;
  int delete_after = 0;
 +int delete_sent_files = 0;
  int only_existing = 0;
  int opt_ignore_existing = 0;
++int need_messages_from_generator = 0;
  int max_delete = 0;
-@@ -92,6 +93,7 @@ int ignore_errors = 0;
+ int ignore_errors = 0;
  int modify_window = 0;
- int blocking_io = -1;
- int checksum_seed = 0;
-+int need_messages_from_generator = 0;
- unsigned int block_size = 0;
-@@ -255,6 +257,7 @@ void usage(enum logcode F)
+@@ -266,6 +268,7 @@ void usage(enum logcode F)
    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");
@@ -93,7 +88,7 @@ command before "make":
    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");
-@@ -305,8 +308,8 @@ void usage(enum logcode F)
+@@ -317,8 +320,8 @@ void usage(enum logcode F)
  }
  
  enum {OPT_VERSION = 1000, OPT_SENDER, OPT_EXCLUDE, OPT_EXCLUDE_FROM,
@@ -101,10 +96,10 @@ command before "make":
 -      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_READ_BATCH, OPT_WRITE_BATCH, OPT_TIMEOUT,
        OPT_REFUSED_BASE = 9000};
  
-@@ -325,6 +328,7 @@ static struct poptOption long_options[] 
+@@ -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 },
@@ -112,7 +107,7 @@ command before "make":
    {"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 },
-@@ -514,6 +518,11 @@ int parse_arguments(int *argc, const cha
+@@ -539,6 +543,11 @@ int parse_arguments(int *argc, const cha
                        delete_mode = 1;
                        break;
  
@@ -122,9 +117,9 @@ command before "make":
 +                      break;
 +
                case OPT_EXCLUDE:
-                       if (am_server || sanitize_paths)
-                               return 0; /* Impossible... */
-@@ -983,6 +992,9 @@ void server_options(char **args,int *arg
+                       add_exclude(&exclude_list, poptGetOptArg(pc), 0);
+                       break;
+@@ -1095,6 +1104,9 @@ void server_options(char **args,int *arg
                }
        }
  
@@ -134,26 +129,26 @@ command before "make":
        *argc = ac;
        return;
  
---- receiver.c 21 May 2004 08:27:04 -0000      1.79
-+++ receiver.c 4 Jun 2004 05:22:00 -0000
-@@ -47,6 +47,7 @@ extern int ignore_errors;
- extern int orig_umask;
- extern int keep_partial;
- extern int checksum_seed;
+--- orig/receiver.c    2004-09-21 09:40:27
++++ receiver.c 2004-08-13 08:38:51
+@@ -45,6 +45,7 @@ extern char *backup_dir;
+ extern char *backup_suffix;
+ extern int backup_suffix_len;
+ extern int cleanup_got_literal;
 +extern int delete_sent_files;
- static void delete_one(char *fn, int is_dir)
- {
-@@ -295,7 +296,7 @@ int recv_files(int f_in,struct file_list
+ extern int module_id;
+ extern int ignore_errors;
+ extern int orig_umask;
+@@ -337,7 +338,7 @@ int recv_files(int f_in, struct file_lis
        char *fname, fbuf[MAXPATHLEN];
        char template[MAXPATHLEN];
        char fnametmp[MAXPATHLEN];
--      char *fnamecmp;
-+      char *fnamecmp, numbuf[4];
+-      char *fnamecmp, *partialptr;
++      char *fnamecmp, *partialptr, numbuf[4];
        char fnamecmpbuf[MAXPATHLEN];
-       struct map_struct *mapbuf;
-       int i;
-@@ -477,16 +478,20 @@ int recv_files(int f_in,struct file_list
+       struct file_struct *file;
+       struct stats initial_stats;
+@@ -572,7 +573,12 @@ int recv_files(int f_in, struct file_lis
  
                cleanup_disable();
  
@@ -164,13 +159,14 @@ command before "make":
 +                              send_msg(MSG_SUCCESS, numbuf, 4);
 +                      }
 +              } else {
-                       if (csum_length == SUM_LENGTH) {
-                               rprintf(FERROR,"ERROR: file corruption in %s. File changed during transfer?\n",
-                                       full_fname(fname));
-                       } else {
+                       int msgtype = csum_length == SUM_LENGTH || read_batch ?
+                               FERROR : FINFO;
+                       if (msgtype == FERROR || verbose) {
+@@ -596,9 +602,8 @@ int recv_files(int f_in, struct file_lis
+                                       keptstr, redostr);
+                       }
+                       if (csum_length != SUM_LENGTH) {
 -                              char buf[4];
-                               if (verbose > 1)
-                                       rprintf(FINFO,"redoing %s(%d)\n",fname,i);
 -                              SIVAL(buf, 0, i);
 -                              send_msg(MSG_REDO, buf, 4);
 +                              SIVAL(numbuf, 0, i);
@@ -178,8 +174,8 @@ command before "make":
                        }
                }
        }
---- rsync.h    16 May 2004 07:28:24 -0000      1.204
-+++ rsync.h    4 Jun 2004 05:22:00 -0000
+--- orig/rsync.h       2004-08-03 15:41:32
++++ 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 */
@@ -188,7 +184,7 @@ command before "make":
  
  /* update this if you make incompatible changes */
  #define PROTOCOL_VERSION 28
-@@ -123,6 +124,7 @@ enum logcode { FERROR=1, FINFO=2, FLOG=3
+@@ -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 {
@@ -196,9 +192,9 @@ command before "make":
        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 */
---- rsync.yo   21 May 2004 09:44:32 -0000      1.170
-+++ rsync.yo   4 Jun 2004 05:22:01 -0000
-@@ -312,6 +312,7 @@ verb(
+--- orig/rsync.yo      2004-09-20 05:10:48
++++ rsync.yo   2004-07-03 20:17:10
+@@ -343,6 +343,7 @@ verb(
       --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
@@ -206,7 +202,7 @@ command before "make":
       --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
-@@ -598,6 +599,11 @@ receiving side before transferring files
+@@ -654,6 +655,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.
  
@@ -218,17 +214,17 @@ command before "make":
  dit(bf(--ignore-errors)) Tells --delete to go ahead and delete files
  even when there are I/O errors.
  
---- sender.c   15 May 2004 19:31:10 -0000      1.40
-+++ sender.c   4 Jun 2004 05:22:01 -0000
-@@ -27,6 +27,7 @@ extern int dry_run;
+--- orig/sender.c      2004-09-20 05:10:48
++++ sender.c   2004-07-26 16:49:19
+@@ -26,6 +26,7 @@ extern int io_error;
+ extern int dry_run;
  extern int am_server;
  extern int am_daemon;
- extern int protocol_version;
 +extern int delete_sent_files;
- /**
-@@ -104,7 +105,29 @@ static struct sum_struct *receive_sums(i
+ extern int protocol_version;
+ extern int make_backups;
+ extern struct stats stats;
+@@ -108,7 +109,29 @@ static struct sum_struct *receive_sums(i
        return s;
  }
  
@@ -258,7 +254,7 @@ command before "make":
  
  void send_files(struct file_list *flist, int f_out, int f_in)
  {
-@@ -129,6 +152,8 @@ void send_files(struct file_list *flist,
+@@ -127,6 +150,8 @@ void send_files(struct file_list *flist,
        if (verbose > 2)
                rprintf(FINFO, "send_files starting\n");
  
@@ -267,13 +263,13 @@ command before "make":
        while (1) {
                unsigned int offset;
  
-@@ -300,6 +325,9 @@ void send_files(struct file_list *flist,
-               if (verbose > 2)
-                       rprintf(FINFO, "sender finished %s\n", fname);
+@@ -257,6 +282,9 @@ void send_files(struct file_list *flist,
+                       rprintf(FINFO, "sender finished %s\n",
+                               safe_fname(fname));
+               }
 +
 +              /* Flag that we actually sent this entry. */
 +              file->flags |= FLAG_SENT;
        }
+       make_backups = save_make_backups;
  
-       if (verbose > 2)