Got rid of support for protocols 17 and 18.

diff --git a/clientserver.c b/clientserver.c
index 5c931fa..0e07da7 100644 (file)
--- a/clientserver.c
+++ b/clientserver.c
@@ -192,7 +192,7 @@ int start_inband_exchange(char *user, char *path, int f_in, int f_out, int argc)
        io_printf(f_out, "\n");
 
        if (protocol_version < 23) {
-               if (protocol_version == 22 || (protocol_version > 17 && !am_sender))
+               if (protocol_version == 22 || !am_sender)
                        io_start_multiplex_in(f_in);
        }
 
@@ -457,7 +457,7 @@ static int rsync_module(int f_in, int f_out, int i)
 #endif
 
        if (protocol_version < 23) {
-               if (protocol_version == 22 || (protocol_version > 17 && am_sender))
+               if (protocol_version == 22 || am_sender)
                        io_start_multiplex_out(f_out);
        }
 

diff --git a/exclude.c b/exclude.c
index e345726..20542a5 100644 (file)
--- a/exclude.c
+++ b/exclude.c
@@ -316,7 +316,6 @@ void add_exclude_file(struct exclude_struct ***listp, const char *fname,
 void send_exclude_list(int f)
 {
        int i;
-       extern int protocol_version;
        extern int list_only, recurse;
 
        /* This is a complete hack - blame Rusty.
@@ -341,10 +340,6 @@ void send_exclude_list(int f)
                l = strlen(pattern);
                if (l == 0) continue;
                if (exclude_list[i]->include) {
-                       if (protocol_version < 19) {
-                               rprintf(FERROR,"remote rsync does not support include syntax - aborting\n");
-                               exit_cleanup(RERR_UNSUPPORTED);
-                       }
                        write_int(f,l+2);
                        write_buf(f,"+ ",2);
                } else {

diff --git a/receiver.c b/receiver.c
index cddad6d..bc297ca 100644 (file)
--- a/receiver.c
+++ b/receiver.c
@@ -23,7 +23,6 @@
 extern int verbose;
 extern int recurse;
 extern int delete_mode;
-extern int protocol_version;
 extern int csum_length;
 extern struct stats stats;
 extern int dry_run;
@@ -41,49 +40,6 @@ extern char *backup_suffix;
 extern int backup_suffix_len;
 extern int cleanup_got_literal;
 
-static struct delete_list {
-       DEV64_T dev;
-       INO64_T inode;
-} *delete_list;
-static int dlist_len, dlist_alloc_len;
-
-/* yuck! This function wouldn't have been necessary if I had the sorting
- * algorithm right. Unfortunately fixing the sorting algorithm would introduce
- * a backward incompatibility as file list indexes are sent over the link.
- */
-static int delete_already_done(struct file_list *flist,int j)
-{
-       int i;
-       STRUCT_STAT st;
-
-       if (link_stat(f_name(flist->files[j]), &st)) return 1;
-
-       for (i = 0; i < dlist_len; i++) {
-               if (st.st_ino == delete_list[i].inode &&
-                   (DEV64_T)st.st_dev == delete_list[i].dev)
-                       return 1;
-       }
-
-       return 0;
-}
-
-static void add_delete_entry(struct file_struct *file)
-{
-       if (dlist_len == dlist_alloc_len) {
-               dlist_alloc_len += 1024;
-               delete_list = realloc_array(delete_list, struct delete_list,
-                                           dlist_alloc_len);
-               if (!delete_list) out_of_memory("add_delete_entry");
-       }
-
-       delete_list[dlist_len].dev = file->dev;
-       delete_list[dlist_len].inode = file->inode;
-       dlist_len++;
-
-       if (verbose > 3)
-               rprintf(FINFO,"added %s to delete list\n", f_name(file));
-}
-
 static void delete_one(char *fn, int is_dir)
 {
        if (!is_dir) {
@@ -138,9 +94,6 @@ void delete_files(struct file_list *flist)
                if (!S_ISDIR(flist->files[j]->mode) ||
                    !(flist->files[j]->flags & FLAG_DELETE)) continue;
 
-               if (protocol_version < 19 &&
-                   delete_already_done(flist, j)) continue;
-
                name = f_name_to(flist->files[j], fbuf, sizeof fbuf);
 
                if (!(local_file_list = send_file_list(-1,1,&name)))
@@ -152,9 +105,6 @@ void delete_files(struct file_list *flist)
                for (i = local_file_list->count-1; i >= 0; i--) {
                        if (max_delete && deletion_count > max_delete) break;
                        if (!local_file_list->files[i]->basename) continue;
-                       if (protocol_version < 19 &&
-                           S_ISDIR(local_file_list->files[i]->mode))
-                               add_delete_entry(local_file_list->files[i]);
                        if (-1 == flist_find(flist,local_file_list->files[i])) {
                                char *f = f_name(local_file_list->files[i]);
                                if (make_backups && (backup_dir || !is_backup_file(f))) {
@@ -251,7 +201,7 @@ static int receive_data(int f_in,struct map_struct *mapbuf,int fd,char *fname,
 
        sum_init();
 
-       for (i=recv_token(f_in,&data); i != 0; i=recv_token(f_in,&data)) {
+       while ((i = recv_token(f_in, &data)) != 0) {
                if (do_progress)
                        show_progress(offset, total_size);
 
@@ -522,11 +472,9 @@ int recv_files(int f_in,struct file_list *flist,char *local_name,int f_gen)
                }
        }
 
-       if (delete_after) {
-               if (recurse && delete_mode && !local_name && flist->count>0) {
-                       delete_files(flist);
-               }
-       }
+       if (delete_after && recurse && delete_mode && !local_name
+           && flist->count > 0)
+               delete_files(flist);
 
        if (preserve_hard_links)
                do_hard_links();

diff --git a/rsync.h b/rsync.h
index 73ba1d9..e89b43c 100644 (file)
--- a/rsync.h
+++ b/rsync.h
@@ -70,7 +70,7 @@
  * unlikely to begin by sending a byte between MIN_PROTOCL_VERSION and
  * MAX_PROTOCOL_VERSION. */
 
-#define MIN_PROTOCOL_VERSION 17
+#define MIN_PROTOCOL_VERSION 19
 #define OLD_PROTOCOL_VERSION 20
 #define MAX_PROTOCOL_VERSION 40