Get rid of some fuzz.
[rsync/rsync-patches.git] / backup-dir-dels.diff
index 4eca0fa..58fc8cf 100644 (file)
@@ -14,8 +14,8 @@ suffix.
 
 Marc St-Onge
 
---- orig/backup.c      2004-09-20 19:50:13
-+++ backup.c   2004-09-22 02:18:49
+--- orig/backup.c      2005-01-19 20:11:10
++++ backup.c   2004-09-22 02:36:06
 @@ -22,11 +22,17 @@
  
  extern int verbose;
@@ -82,7 +82,7 @@ Marc St-Onge
        char *end = rel + strlen(rel);
        char *p = end;
  
-@@ -173,7 +199,8 @@ static int keep_backup(char *fname)
+@@ -168,7 +194,8 @@ static int keep_backup(char *fname)
        if (!(file = make_file(fname, NULL, NO_EXCLUDES)))
                return 1; /* the file could have disappeared */
  
@@ -91,31 +91,24 @@ Marc St-Onge
 +      if (!buf)
                return 0;
  
- #ifdef HAVE_MKNOD
-@@ -262,7 +289,18 @@ static int keep_backup(char *fname)
- /* main backup switch routine */
- int make_backup(char *fname)
- {
-+      int ret;
-       if (backup_dir)
--              return keep_backup(fname);
--      return make_simple_backup(fname);
-+              ret = keep_backup(fname);
-+      else
-+              ret = make_simple_backup(fname);
-+      deleting = 0;   /* Always restore the default backup process. */
-+      return ret;
-+}
+       /* Check to see if this is a device file, or link */
+@@ -259,3 +286,13 @@ int make_backup(char *fname)
+               return keep_backup(fname);
+       return make_simple_backup(fname);
+ }
 +
 +/* backup switch routine called only when backing-up deleted file */
 +int safe_delete(char *fname)
 +{
++      int ret;
 +      deleting = 1;
-+      return make_backup(fname);
- }
---- orig/options.c     2004-09-20 05:10:48
-+++ options.c  2004-09-22 02:25:03
-@@ -113,10 +113,14 @@ int no_detach = 0;
++      ret = make_backup(fname);
++      deleting = 0;
++      return ret;
++}
+--- orig/options.c     2005-01-20 23:05:34
++++ options.c  2004-11-27 18:37:18
+@@ -116,10 +116,14 @@ int no_detach = 0;
  int write_batch = 0;
  int read_batch = 0;
  int backup_dir_len = 0;
@@ -129,18 +122,18 @@ Marc St-Onge
 +char *delete_suffix = NULL;
  char *tmpdir = NULL;
  char *partial_dir = NULL;
- char *compare_dest = NULL;
-@@ -126,7 +130,9 @@ char *log_format = NULL;
+ char *basis_dir[MAX_BASIS_DIRS+1];
+@@ -129,7 +133,9 @@ char *log_format = NULL;
  char *password_file = NULL;
  char *rsync_path = RSYNC_PATH;
  char *backup_dir = NULL;
 +char *delete_dir = NULL;
  char backup_dir_buf[MAXPATHLEN];
 +char delete_dir_buf[MAXPATHLEN];
- int rsync_port = RSYNC_PORT;
- int link_dest = 0;
-@@ -239,7 +245,9 @@ void usage(enum logcode F)
+ int rsync_port = 0;
+ int compare_dest = 0;
+ int copy_dest = 0;
+@@ -247,7 +253,9 @@ void usage(enum logcode F)
    rprintf(F,"     --no-implied-dirs       don't send implied dirs with -R\n");
    rprintf(F," -b, --backup                make backups (see --suffix & --backup-dir)\n");
    rprintf(F,"     --backup-dir            make backups into this directory\n");
@@ -148,26 +141,26 @@ Marc St-Onge
    rprintf(F,"     --suffix=SUFFIX         backup suffix (default %s w/o --backup-dir)\n",BACKUP_SUFFIX);
 +  rprintf(F,"     --delete-suffix=SUFFIX  deleted files suffix (default %s w/o --backup-dir)\n",BACKUP_SUFFIX);
    rprintf(F," -u, --update                update only (don't overwrite newer files)\n");
-   rprintf(F,"     --inplace               update destination files inplace (SEE MAN PAGE)\n");
+   rprintf(F,"     --inplace               update destination files in-place (SEE MAN PAGE)\n");
    rprintf(F," -K, --keep-dirlinks         treat symlinked dir on receiver as dir\n");
-@@ -326,6 +334,7 @@ static struct poptOption long_options[] 
+@@ -331,6 +339,7 @@ static struct poptOption long_options[] 
    /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
-   {"version",          0,  POPT_ARG_NONE,   0,              OPT_VERSION, 0, 0},
+   {"version",          0,  POPT_ARG_NONE,   0, OPT_VERSION, 0, 0},
    {"suffix",           0,  POPT_ARG_STRING, &backup_suffix, 0, 0, 0 },
 +  {"delete-suffix",    0,  POPT_ARG_STRING, &delete_suffix, 0, 0, 0 },
    {"rsync-path",       0,  POPT_ARG_STRING, &rsync_path, 0, 0, 0 },
    {"password-file",    0,  POPT_ARG_STRING, &password_file, 0, 0, 0 },
    {"ignore-times",    'I', POPT_ARG_NONE,   &ignore_times, 0, 0, 0 },
-@@ -396,6 +405,7 @@ static struct poptOption long_options[] 
+@@ -399,6 +408,7 @@ static struct poptOption long_options[] 
+   {"log-format",       0,  POPT_ARG_STRING, &log_format, 0, 0, 0 },
    {"bwlimit",          0,  POPT_ARG_INT,    &bwlimit, 0, 0, 0 },
-   {"address",          0,  POPT_ARG_STRING, &bind_address, 0, 0, 0 },
    {"backup-dir",       0,  POPT_ARG_STRING, &backup_dir, 0, 0, 0 },
 +  {"delete-dir",       0,  POPT_ARG_STRING, &delete_dir, 0, 0, 0 },
    {"hard-links",      'H', POPT_ARG_NONE,   &preserve_hard_links, 0, 0, 0 },
-   {"read-batch",       0,  POPT_ARG_STRING, &batch_name,  OPT_READ_BATCH, 0, 0 },
-   {"write-batch",      0,  POPT_ARG_STRING, &batch_name,  OPT_WRITE_BATCH, 0, 0 },
-@@ -733,6 +743,8 @@ int parse_arguments(int *argc, const cha
-                       compare_dest = sanitize_path(NULL, compare_dest, NULL, 0);
+   {"read-batch",       0,  POPT_ARG_STRING, &batch_name, OPT_READ_BATCH, 0, 0 },
+   {"write-batch",      0,  POPT_ARG_STRING, &batch_name, OPT_WRITE_BATCH, 0, 0 },
+@@ -871,6 +881,8 @@ int parse_arguments(int *argc, const cha
+                       partial_dir = sanitize_path(NULL, partial_dir, NULL, 0);
                if (backup_dir)
                        backup_dir = sanitize_path(NULL, backup_dir, NULL, 0);
 +              if (delete_dir)                                                 
@@ -175,7 +168,7 @@ Marc St-Onge
                if (files_from)
                        files_from = sanitize_path(NULL, files_from, NULL, 0);
        }
-@@ -758,6 +770,12 @@ int parse_arguments(int *argc, const cha
+@@ -897,6 +909,12 @@ int parse_arguments(int *argc, const cha
                        if (check_exclude(elp, backup_dir, 1) < 0)
                                goto options_rejected;
                }
@@ -188,7 +181,7 @@ Marc St-Onge
        }
        if (server_exclude_list.head && files_from) {
                clean_fname(files_from, 1);
-@@ -784,6 +802,16 @@ int parse_arguments(int *argc, const cha
+@@ -917,6 +935,16 @@ int parse_arguments(int *argc, const cha
                        backup_suffix);
                return 0;
        }
@@ -205,7 +198,7 @@ Marc St-Onge
        if (backup_dir) {
                backup_dir_len = strlcpy(backup_dir_buf, backup_dir, sizeof backup_dir_buf);
                backup_dir_remainder = sizeof backup_dir_buf - backup_dir_len;
-@@ -803,6 +831,31 @@ int parse_arguments(int *argc, const cha
+@@ -936,6 +964,31 @@ int parse_arguments(int *argc, const cha
                        "--suffix cannot be a null string without --backup-dir\n");
                return 0;
        }
@@ -237,7 +230,7 @@ Marc St-Onge
  
        if (do_progress && !verbose)
                verbose = 1;
-@@ -1005,6 +1058,10 @@ void server_options(char **args,int *arg
+@@ -1144,6 +1197,10 @@ void server_options(char **args,int *arg
                args[ac++] = "--backup-dir";
                args[ac++] = backup_dir;
        }
@@ -248,7 +241,7 @@ Marc St-Onge
  
        /* Only send --suffix if it specifies a non-default value. */
        if (strcmp(backup_suffix, backup_dir ? "" : BACKUP_SUFFIX) != 0) {
-@@ -1013,7 +1070,13 @@ void server_options(char **args,int *arg
+@@ -1152,7 +1209,13 @@ void server_options(char **args,int *arg
                        goto oom;
                args[ac++] = arg;
        }
@@ -263,9 +256,9 @@ Marc St-Onge
        if (am_sender) {
                if (delete_excluded)
                        args[ac++] = "--delete-excluded";
---- orig/receiver.c    2004-09-21 09:40:27
-+++ receiver.c 2004-09-22 02:09:20
-@@ -42,8 +42,11 @@ extern char *compare_dest;
+--- orig/receiver.c    2005-01-21 00:35:26
++++ receiver.c 2004-11-03 23:10:06
+@@ -44,8 +44,11 @@ extern int basis_dir_cnt;
  extern int make_backups;
  extern int do_progress;
  extern char *backup_dir;
@@ -277,9 +270,9 @@ Marc St-Onge
  extern int cleanup_got_literal;
  extern int module_id;
  extern int ignore_errors;
-@@ -77,11 +80,14 @@ static void delete_one(char *fn, int is_
-       }
- }
+@@ -56,11 +59,14 @@ extern int inplace;
+ extern struct exclude_list_struct server_exclude_list;
  
 -
 +/* Function now checks if file matches backup or delete suffix patterns */
@@ -294,14 +287,15 @@ Marc St-Onge
  }
  
  
-@@ -122,10 +128,11 @@ void delete_files(struct file_list *flis
-                               continue;
+@@ -102,11 +108,12 @@ void delete_files(struct file_list *flis
                        if (flist_find(flist,local_file_list->files[i]) < 0) {
                                char *f = f_name(local_file_list->files[i]);
--                              if (make_backups && (backup_dir || !is_backup_file(f))) {
--                                      make_backup(f);
+                               int mode = local_file_list->files[i]->mode;
+-                              if (make_backups && (backup_dir || !is_backup_file(f))
 +                              int backup_file = is_backup_file(f);
-+                              if (make_backups && (delete_dir || !backup_file)) {
++                              if (make_backups && (delete_dir || !backup_file)
+                                 && !S_ISDIR(mode)) {
+-                                      make_backup(f);
 +                                      safe_delete(f);
                                        if (verbose) {
 -                                              rprintf(FINFO, "deleting %s\n",