Moved one fsync() call.
[rsync-patches.git] / fsync.diff
index 96c3e217df3a62f300ffc4e8f59279e810511fd0..276c454f8a779e5cce34b10e1713b63d89a6b158 100644 (file)
@@ -1,33 +1,39 @@
 This patch from Sami Farin lets you specify --fsync if you want fsync()
 to be called on every file we write.
 
---- orig/options.c     2005-02-25 18:44:31
-+++ options.c  2005-01-28 19:30:47
-@@ -39,6 +39,7 @@ int make_backups = 0;
-  **/
- int whole_file = -1;
-+int do_fsync = 0;
- int archive_mode = 0;
+To use this patch, run these commands for a successful build:
+
+    patch -p1 <patches/fsync.diff
+    ./configure                         (optional if already run)
+    make
+
+--- old/options.c
++++ new/options.c
+@@ -44,6 +44,7 @@ int append_mode = 0;
  int keep_dirlinks = 0;
+ int copy_dirlinks = 0;
  int copy_links = 0;
-@@ -309,6 +310,7 @@ void usage(enum logcode F)
-   rprintf(F,"     --partial               keep partially transferred files\n");
++int do_fsync = 0;
+ int preserve_links = 0;
+ int preserve_hard_links = 0;
+ int preserve_acls = 0;
+@@ -345,6 +346,7 @@ void usage(enum logcode F)
    rprintf(F,"     --partial-dir=DIR       put a partially transferred file into DIR\n");
    rprintf(F,"     --delay-updates         put all updated files into place at transfer's end\n");
+   rprintf(F," -m, --prune-empty-dirs      prune empty directory chains from the file-list\n");
 +  rprintf(F,"     --fsync                 fsync every written file\n");
    rprintf(F,"     --numeric-ids           don't map uid/gid values by user/group name\n");
    rprintf(F,"     --timeout=TIME          set I/O timeout in seconds\n");
    rprintf(F," -I, --ignore-times          don't skip files that match in size and mod-time\n");
-@@ -389,6 +391,7 @@ static struct poptOption long_options[] 
-   {"safe-links",       0,  POPT_ARG_NONE,   &safe_symlinks, 0, 0, 0 },
-   {"help",            'h', POPT_ARG_NONE,   0, 'h', 0, 0 },
-   {"backup",          'b', POPT_ARG_NONE,   &make_backups, 0, 0, 0 },
+@@ -539,6 +541,7 @@ static struct poptOption long_options[] 
+   {"only-write-batch", 0,  POPT_ARG_STRING, &batch_name, OPT_ONLY_WRITE_BATCH, 0, 0 },
+   {"files-from",       0,  POPT_ARG_STRING, &files_from, 0, 0, 0 },
+   {"from0",           '0', POPT_ARG_NONE,   &eol_nulls, 0, 0, 0},
 +  {"fsync",            0,  POPT_ARG_NONE,   &do_fsync, 0, 0, 0 },
-   {"dry-run",         'n', POPT_ARG_NONE,   &dry_run, 0, 0, 0 },
-   {"sparse",          'S', POPT_ARG_NONE,   &sparse_files, 0, 0, 0 },
-   {"cvs-exclude",     'C', POPT_ARG_NONE,   &cvs_exclude, 0, 0, 0 },
-@@ -1421,6 +1424,9 @@ void server_options(char **args,int *arg
+   {"numeric-ids",      0,  POPT_ARG_NONE,   &numeric_ids, 0, 0, 0 },
+   {"timeout",          0,  POPT_ARG_INT,    &io_timeout, 0, 0, 0 },
+   {"rsh",             'e', POPT_ARG_STRING, &shell_cmd, 0, 0, 0 },
+@@ -1777,6 +1780,9 @@ void server_options(char **args,int *arg
                args[ac++] = tmpdir;
        }
  
@@ -37,17 +43,17 @@ to be called on every file we write.
        if (basis_dir[0] && am_sender) {
                /* the server only needs this option if it is not the sender,
                 *   and it may be an older version that doesn't know this
---- orig/receiver.c    2005-02-26 19:27:54
-+++ receiver.c 2005-02-20 00:17:37
-@@ -37,6 +37,7 @@ extern int keep_dirlinks;
+--- old/receiver.c
++++ new/receiver.c
+@@ -37,6 +37,7 @@ extern int protocol_version;
+ extern int relative_paths;
  extern int preserve_hard_links;
  extern int preserve_perms;
- extern int io_error;
 +extern int do_fsync;
  extern int basis_dir_cnt;
  extern int make_backups;
  extern int cleanup_got_literal;
-@@ -289,6 +290,12 @@ static int receive_data(int f_in, char *
+@@ -257,6 +258,12 @@ static int receive_data(int f_in, char *
                exit_cleanup(RERR_FILEIO);
        }
  
@@ -57,31 +63,31 @@ to be called on every file we write.
 +              exit_cleanup(RERR_FILEIO);
 +      }
 +
-       sum_end(file_sum1);
+       sum_len = sum_end(file_sum1);
  
        if (mapbuf)
---- orig/t_stub.c      2005-01-25 12:14:14
-+++ t_stub.c   2004-10-09 04:03:09
-@@ -28,6 +28,7 @@
+--- old/t_stub.c
++++ new/t_stub.c
+@@ -21,6 +21,7 @@
+ #include "rsync.h"
  
++int do_fsync = 0;
  int modify_window = 0;
  int module_id = -1;
-+int do_fsync;
- char *partial_dir;
- struct filter_list_struct server_filter_list;
---- orig/util.c        2005-02-23 02:57:27
-+++ util.c     2004-07-03 20:18:13
-@@ -33,6 +33,7 @@ extern int module_id;
- extern int modify_window;
- extern char *partial_dir;
- extern struct filter_list_struct server_filter_list;
+ int relative_paths = 0;
+--- old/util.c
++++ new/util.c
+@@ -25,6 +25,7 @@
+ extern int verbose;
+ extern int dry_run;
+ extern int module_id;
 +extern int do_fsync;
int sanitize_paths = 0;
-@@ -298,6 +299,12 @@ int copy_file(char *source, char *dest, 
-               return -1;
+ extern int modify_window;
extern int relative_paths;
+ extern int human_readable;
+@@ -309,6 +310,12 @@ int copy_file(const char *source, const 
+                       full_fname(source));
        }
  
 +      if (do_fsync && fsync(ofd) < 0) {
@@ -90,6 +96,6 @@ to be called on every file we write.
 +              return -1;
 +      }
 +
-       return 0;
- }
+       if (close(ofd) < 0) {
+               rsyserr(FERROR, errno, "close failed on %s",
+                       full_fname(dest));