X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/a75921111fdf7354d07296084231945f01b3c644..fc068916893d69f75d9eab25b5ebd46ed5341683:/detect-renamed.diff diff --git a/detect-renamed.diff b/detect-renamed.diff index 6c300b1..984694e 100644 --- a/detect-renamed.diff +++ b/detect-renamed.diff @@ -34,7 +34,7 @@ TODO: --- old/flist.c +++ new/flist.c -@@ -55,6 +55,7 @@ extern int non_perishable_cnt; +@@ -56,6 +56,7 @@ extern int non_perishable_cnt; extern int prune_empty_dirs; extern int copy_links; extern int copy_unsafe_links; @@ -42,7 +42,7 @@ TODO: extern int protocol_version; extern int sanitize_paths; extern struct stats stats; -@@ -80,6 +81,8 @@ static int64 tmp_dev, tmp_ino; +@@ -86,6 +87,8 @@ static int64 tmp_dev, tmp_ino; #endif static char tmp_sum[MD4_SUM_LENGTH]; @@ -51,7 +51,7 @@ TODO: static char empty_sum[MD4_SUM_LENGTH]; static int flist_count_offset; /* for --delete --progress */ -@@ -262,6 +265,45 @@ static mode_t from_wire_mode(int mode) +@@ -268,6 +271,45 @@ static mode_t from_wire_mode(int mode) return mode; } @@ -94,10 +94,10 @@ TODO: + return u_strcmp(f1->dirname, f2->dirname); +} + - static void send_directory(int f, struct file_list *flist, - char *fbuf, int len); + static void send_directory(int f, struct file_list *flist, int ndx, + char *fbuf, int len, int flags); -@@ -1501,6 +1543,25 @@ struct file_list *recv_file_list(int f) +@@ -1702,6 +1744,25 @@ struct file_list *recv_file_list(int f) clean_flist(flist, relative_paths, 1); @@ -110,7 +110,7 @@ TODO: + memcpy(the_fattr_list.files, flist->files, + j * sizeof (struct file_struct *)); + qsort(the_fattr_list.files, j, -+ sizeof the_fattr_list.files[0], (int (*)())fattr_compare); ++ sizeof the_fattr_list.files[0], (int (*)())fattr_compare); + the_fattr_list.low = 0; + while (j-- > 0) { + struct file_struct *fp = the_fattr_list.files[j]; @@ -120,12 +120,12 @@ TODO: + the_fattr_list.high = j; + } + - if (f >= 0) { - recv_uid_list(f, flist); - + if (incremental) { + qsort(dir_flist->files + dstart, dir_flist->count - dstart, + sizeof dir_flist->files[0], (int (*)())file_compare); --- old/generator.c +++ new/generator.c -@@ -76,6 +76,7 @@ extern char *basis_dir[]; +@@ -79,6 +79,7 @@ extern char *basis_dir[]; extern int compare_dest; extern int copy_dest; extern int link_dest; @@ -133,17 +133,17 @@ TODO: extern int whole_file; extern int list_only; extern int new_root_dir; -@@ -91,6 +92,7 @@ extern char *backup_dir; - extern char *backup_suffix; +@@ -95,6 +96,7 @@ extern char *backup_suffix; extern int backup_suffix_len; - extern struct file_list *the_file_list; -+extern struct file_list the_fattr_list; + extern struct file_list *cur_flist, *first_flist, *dir_flist; extern struct filter_list_struct server_filter_list; ++extern struct file_list the_fattr_list; int ignore_perishable = 0; -@@ -98,12 +100,14 @@ int non_perishable_cnt = 0; - int maybe_ATTRS_REPORT = 0; + int non_perishable_cnt = 0; +@@ -102,12 +104,14 @@ int maybe_ATTRS_REPORT = 0; + static dev_t dev_zero; static int deletion_count = 0; /* used to implement --max-delete */ +static int unexplored_dirs = 1; static int deldelay_size = 0, deldelay_cnt = 0; @@ -157,7 +157,7 @@ TODO: #define DEL_RECURSE (1<<1) /* recurse */ #define DEL_DIR_IS_EMPTY (1<<2) /* internal delete_FUNCTIONS use only */ -@@ -125,11 +129,120 @@ static int is_backup_file(char *fn) +@@ -129,11 +133,120 @@ static int is_backup_file(char *fn) return k > 0 && strcmp(fn+k, backup_suffix) == 0; } @@ -278,7 +278,7 @@ TODO: */ static enum delret delete_item(char *fbuf, int mode, char *replace, int flags) { -@@ -151,6 +264,8 @@ static enum delret delete_item(char *fbu +@@ -155,6 +268,8 @@ static enum delret delete_item(char *fbu goto check_ret; /* OK: try to delete the directory. */ } @@ -287,7 +287,7 @@ TODO: if (!replace && max_delete >= 0 && ++deletion_count > max_delete) return DR_AT_LIMIT; -@@ -197,6 +312,8 @@ static enum delret delete_item(char *fbu +@@ -201,6 +316,8 @@ static enum delret delete_item(char *fbu * its contents, otherwise just checks for content. Returns DR_SUCCESS or * DR_NOT_EMPTY. Note that fname must point to a MAXPATHLEN buffer! (The * buffer is used for recursion, but returned unchanged.) @@ -296,7 +296,7 @@ TODO: */ static enum delret delete_dir_contents(char *fname, int flags) { -@@ -216,7 +333,9 @@ static enum delret delete_dir_contents(c +@@ -220,7 +337,9 @@ static enum delret delete_dir_contents(c save_filters = push_local_filters(fname, dlen); non_perishable_cnt = 0; @@ -306,7 +306,7 @@ TODO: ret = non_perishable_cnt ? DR_NOT_EMPTY : DR_SUCCESS; if (!dirlist->count) -@@ -253,6 +372,8 @@ static enum delret delete_dir_contents(c +@@ -257,6 +376,8 @@ static enum delret delete_dir_contents(c if (S_ISDIR(fp->mode) && delete_dir_contents(fname, flags | DEL_RECURSE) != DR_SUCCESS) ret = DR_NOT_EMPTY; @@ -315,7 +315,7 @@ TODO: if (delete_item(fname, fp->mode, NULL, flags) != DR_SUCCESS) ret = DR_NOT_EMPTY; } -@@ -405,15 +526,19 @@ static void do_delayed_deletions(char *d +@@ -409,13 +530,17 @@ static void do_delayed_deletions(char *d * all the --delete-WHEN options. Note that the fbuf pointer must point to a * MAXPATHLEN buffer with the name of the directory in it (the functions we * call will append names onto the end, but the old dir value will be restored @@ -325,11 +325,9 @@ TODO: + * Note: --detect-rename may use this routine with DEL_NO_DELETIONS set! + */ static void delete_in_dir(struct file_list *flist, char *fbuf, -- struct file_struct *file, STRUCT_STAT *stp) -+ struct file_struct *file, STRUCT_STAT *stp, int flags) +- struct file_struct *file, dev_t *fs_dev) ++ struct file_struct *file, dev_t *fs_dev, int flags) { - static int min_depth = MAXPATHLEN, cur_depth = -1; - static void *filt_array[MAXPATHLEN/2+1]; static int already_warned = 0; struct file_list *dirlist; - char delbuf[MAXPATHLEN]; @@ -338,7 +336,7 @@ TODO: int dlen, i; if (!flist) { -@@ -427,6 +552,8 @@ static void delete_in_dir(struct file_li +@@ -426,21 +551,28 @@ static void delete_in_dir(struct file_li if (verbose > 2) rprintf(FINFO, "delete_in_dir(%s)\n", fbuf); @@ -347,9 +345,6 @@ TODO: if (allowed_lull) maybe_send_keepalive(); -@@ -434,12 +561,14 @@ static void delete_in_dir(struct file_li - return; /* Impossible... */ - if (io_error && !ignore_errors) { - if (already_warned) + if (!already_warned) { @@ -366,18 +361,16 @@ TODO: + flags |= DEL_NO_DELETIONS; } - while (cur_depth >= F_DEPTH(file) && cur_depth >= min_depth) -@@ -450,6 +579,9 @@ static void delete_in_dir(struct file_li dlen = strlen(fbuf); - filt_array[cur_depth] = push_local_filters(fbuf, dlen); + change_local_filter_dir(fbuf, dlen, F_DEPTH(file)); + if (detect_renamed) + unexplored_dirs--; + if (one_file_system) { if (file->flags & FLAG_TOP_DIR) - filesystem_dev = stp->st_dev; -@@ -459,6 +591,11 @@ static void delete_in_dir(struct file_li + filesystem_dev = *fs_dev; +@@ -450,6 +582,11 @@ static void delete_in_dir(struct file_li dirlist = get_dirlist(fbuf, dlen, 0); @@ -389,7 +382,7 @@ TODO: /* If an item in dirlist is not found in flist, delete it * from the filesystem. */ for (i = dirlist->count; i--; ) { -@@ -471,16 +608,23 @@ static void delete_in_dir(struct file_li +@@ -462,16 +599,23 @@ static void delete_in_dir(struct file_li f_name(fp, NULL)); continue; } @@ -416,42 +409,42 @@ TODO: flist_free(dirlist); } -@@ -510,9 +654,9 @@ static void do_delete_pass(struct file_l +@@ -501,9 +645,9 @@ static void do_delete_pass(struct file_l || !S_ISDIR(st.st_mode)) continue; -- delete_in_dir(flist, fbuf, file, &st); -+ delete_in_dir(flist, fbuf, file, &st, 0); +- delete_in_dir(flist, fbuf, file, &st.st_dev); ++ delete_in_dir(flist, fbuf, file, &st.st_dev, 0); } -- delete_in_dir(NULL, NULL, NULL, NULL); -+ delete_in_dir(NULL, NULL, NULL, NULL, 0); +- delete_in_dir(NULL, NULL, NULL, &dev_zero); ++ delete_in_dir(NULL, NULL, NULL, &dev_zero, 0); if (do_progress && !am_server) rprintf(FINFO, " \r"); -@@ -1047,6 +1191,7 @@ static int try_dests_non(struct file_str +@@ -1038,6 +1182,7 @@ static int try_dests_non(struct file_str return j; } +static struct bitbag *delayed_bits = NULL; static int phase = 0; - /* Acts on the_file_list->file's ndx'th item, whose name is fname. If a dir, -@@ -1232,8 +1377,12 @@ static void recv_generator(char *fname, - if (real_ret != 0 && one_file_system) - real_st.st_dev = filesystem_dev; - if (delete_during && f_out != -1 && !phase && dry_run < 2 + /* Acts on cur_flist->file's ndx'th item, whose name is fname. If a dir, +@@ -1229,8 +1374,12 @@ static void recv_generator(char *fname, + } + } + else if (delete_during && f_out != -1 && !phase && dry_run < 2 - && (file->flags & FLAG_XFER_DIR)) -- delete_in_dir(the_file_list, fname, file, &real_st); +- delete_in_dir(cur_flist, fname, file, &real_st.st_dev); + && (file->flags & FLAG_XFER_DIR)) { + if (detect_renamed && real_ret != 0) + unexplored_dirs++; -+ delete_in_dir(the_file_list, fname, file, &real_st, ++ delete_in_dir(cur_flist, fname, file, &real_st.st_dev, + delete_during < 0 ? DEL_NO_DELETIONS : 0); + } return; } -@@ -1497,8 +1646,14 @@ static void recv_generator(char *fname, +@@ -1494,8 +1643,14 @@ static void recv_generator(char *fname, if (preserve_hard_links && F_HLINK_NOT_LAST(file)) return; #endif @@ -467,9 +460,9 @@ TODO: rsyserr(FERROR, stat_errno, "recv_generator: failed to stat %s", full_fname(fname)); return; -@@ -1688,6 +1843,12 @@ void generate_files(int f_out, struct fi - (long)getpid(), flist->count); - } +@@ -1683,6 +1838,12 @@ void generate_files(int f_out, char *loc + if (verbose > 2) + rprintf(FINFO, "generator starting pid=%ld\n", (long)getpid()); + if (detect_renamed) { + delayed_bits = bitbag_create(flist->count); @@ -477,42 +470,50 @@ TODO: + delete_during = -1; + } + - if (delete_before && !local_name && flist->count > 0) - do_delete_pass(flist); + if (delete_before && !local_name && cur_flist->count > 0) + do_delete_pass(cur_flist); if (delete_during == 2) { -@@ -1698,7 +1859,7 @@ void generate_files(int f_out, struct fi +@@ -1693,7 +1854,7 @@ void generate_files(int f_out, char *loc } do_progress = 0; -- if (append_mode || whole_file < 0) -+ if (append_mode || detect_renamed || whole_file < 0) +- if (append_mode > 0 || whole_file < 0) ++ if (append_mode > 0 || detect_renamed || whole_file < 0) whole_file = 0; if (verbose >= 2) { rprintf(FINFO, "delta-transmission %s\n", -@@ -1754,7 +1915,22 @@ void generate_files(int f_out, struct fi - } - recv_generator(NULL, NULL, 0, 0, code, -1); - if (delete_during) -- delete_in_dir(NULL, NULL, NULL, NULL); -+ delete_in_dir(NULL, NULL, NULL, NULL, 0); -+ -+ if (detect_renamed) { -+ if (delete_during < 0) -+ delete_during = 0; -+ detect_renamed = 0; +@@ -1718,7 +1879,7 @@ void generate_files(int f_out, char *loc + dirdev = MAKEDEV(DEV_MAJOR(devp), DEV_MINOR(devp)); + } else + dirdev = MAKEDEV(0, 0); +- delete_in_dir(cur_flist, f_name(fp, fbuf), fp, &dirdev); ++ delete_in_dir(cur_flist, f_name(fp, fbuf), fp, &dirdev, 0); + } + } + for (i = cur_flist->low; i <= cur_flist->high; i++) { +@@ -1764,7 +1925,21 @@ void generate_files(int f_out, char *loc + + if (!incremental) { + if (delete_during) +- delete_in_dir(NULL, NULL, NULL, &dev_zero); ++ delete_in_dir(NULL, NULL, NULL, &dev_zero, 0); ++ if (detect_renamed) { ++ if (delete_during < 0) ++ delete_during = 0; ++ detect_renamed = 0; + -+ for (i = -1; (i = bitbag_next_bit(delayed_bits, i)) >= 0; ) { -+ struct file_struct *file = flist->files[i]; -+ if (local_name) -+ strlcpy(fbuf, local_name, sizeof fbuf); -+ else -+ f_name(file, fbuf); -+ recv_generator(fbuf, file, i, itemizing, code, f_out); -+ } -+ } - - phase++; - csum_length = SUM_LENGTH; ++ for (i = -1; (i = bitbag_next_bit(delayed_bits, i)) >= 0; ) { ++ struct file_struct *file = flist->files[i]; ++ if (local_name) ++ strlcpy(fbuf, local_name, sizeof fbuf); ++ else ++ f_name(file, fbuf); ++ recv_generator(fbuf, file, i, itemizing, code, f_out); ++ } ++ } + phase++; + if (verbose > 2) { + rprintf(FINFO, "generate_files phase=%d\n", --- old/options.c +++ new/options.c @@ -78,6 +78,7 @@ int am_generator = 0; @@ -523,7 +524,7 @@ TODO: int numeric_ids = 0; int allow_8bit_chars = 0; int force_delete = 0; -@@ -342,6 +343,7 @@ void usage(enum logcode F) +@@ -343,6 +344,7 @@ void usage(enum logcode F) rprintf(F," --modify-window=NUM compare mod-times with reduced accuracy\n"); rprintf(F," -T, --temp-dir=DIR create temporary files in directory DIR\n"); rprintf(F," -y, --fuzzy find similar file for basis if no dest file\n"); @@ -531,7 +532,7 @@ TODO: rprintf(F," --compare-dest=DIR also compare destination files relative to DIR\n"); rprintf(F," --copy-dest=DIR ... and include copies of unchanged files\n"); rprintf(F," --link-dest=DIR hardlink to files in DIR when unchanged\n"); -@@ -496,6 +498,7 @@ static struct poptOption long_options[] +@@ -497,6 +499,7 @@ static struct poptOption long_options[] {"compare-dest", 0, POPT_ARG_STRING, 0, OPT_COMPARE_DEST, 0, 0 }, {"copy-dest", 0, POPT_ARG_STRING, 0, OPT_COPY_DEST, 0, 0 }, {"link-dest", 0, POPT_ARG_STRING, 0, OPT_LINK_DEST, 0, 0 }, @@ -539,7 +540,7 @@ TODO: {"fuzzy", 'y', POPT_ARG_NONE, &fuzzy_basis, 0, 0, 0 }, {"compress", 'z', POPT_ARG_NONE, 0, 'z', 0, 0 }, {"compress-level", 0, POPT_ARG_INT, &def_compress_level, 'z', 0, 0 }, -@@ -1355,7 +1358,7 @@ int parse_arguments(int *argc, const cha +@@ -1368,7 +1371,7 @@ int parse_arguments(int *argc, const cha inplace = 1; } @@ -548,7 +549,7 @@ TODO: partial_dir = tmp_partialdir; if (inplace) { -@@ -1364,6 +1367,7 @@ int parse_arguments(int *argc, const cha +@@ -1377,6 +1380,7 @@ int parse_arguments(int *argc, const cha snprintf(err_buf, sizeof err_buf, "--%s cannot be used with --%s\n", append_mode ? "append" : "inplace", @@ -556,7 +557,7 @@ TODO: delay_updates ? "delay-updates" : "partial-dir"); return 0; } -@@ -1674,6 +1678,8 @@ void server_options(char **args,int *arg +@@ -1690,6 +1694,8 @@ void server_options(char **args,int *arg args[ac++] = "--super"; if (size_only) args[ac++] = "--size-only";