make proto
---- orig/io.c 2004-06-23 01:13:56
-+++ io.c 2004-07-03 20:17:10
-@@ -225,6 +225,14 @@ static void read_msg_fd(void)
+--- orig/generator.c 2005-02-16 15:35:52
++++ generator.c 2005-02-15 21:33:51
+@@ -38,6 +38,7 @@ extern int preserve_gid;
+ extern int preserve_times;
+ extern int omit_dir_times;
+ extern int delete_during;
++extern int remove_sent_files;
+ extern int update_only;
+ extern int opt_ignore_existing;
+ extern int inplace;
+@@ -510,6 +511,11 @@ static void recv_generator(char *fname,
+ rprintf(FINFO, "%s -> %s\n", safe_fname(fname),
+ safe_fname(file->u.link));
+ }
++ if (remove_sent_files && !dry_run) {
++ char numbuf[4];
++ SIVAL(numbuf, 0, ndx);
++ io_multiplex_write(MSG_SUCCESS, numbuf, 4);
++ }
+ }
+ #endif
+ return;
+--- orig/io.c 2005-02-03 02:04:20
++++ io.c 2005-02-15 21:33:51
+@@ -244,6 +244,14 @@ static void read_msg_fd(void)
read_loop(fd, buf, 4);
redo_list_add(IVAL(buf,0));
break;
case MSG_INFO:
case MSG_ERROR:
case MSG_LOG:
-@@ -634,6 +642,16 @@ static int readfd_unbuffered(int fd, cha
- read_loop(fd, buffer, remaining);
- bufferIdx = 0;
+@@ -677,6 +685,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) {
case MSG_INFO:
case MSG_ERROR:
if (remaining >= sizeof line) {
---- orig/main.c 2004-06-30 07:24:30
-+++ main.c 2004-07-03 20:17:10
-@@ -42,6 +42,7 @@ extern int list_only;
- extern int local_server;
+--- orig/main.c 2005-02-15 19:27:04
++++ main.c 2005-02-15 21:33:51
+@@ -33,12 +33,14 @@ extern int verbose;
+ extern int itemize_changes;
+ extern int blocking_io;
+ extern int delete_before;
++extern int remove_sent_files;
+ extern int daemon_over_rsh;
+ extern int do_stats;
+ extern int dry_run;
+ extern int list_only;
extern int log_got_error;
extern int module_id;
+extern int need_messages_from_generator;
extern int orig_umask;
+ extern int copy_links;
extern int keep_dirlinks;
- extern int preserve_hard_links;
-@@ -585,6 +586,8 @@ void start_server(int f_in, int f_out, i
+@@ -442,6 +444,12 @@ static void do_server_sender(int f_in, i
+ exit_cleanup(RERR_SYNTAX);
+ return;
+ }
++ if (am_daemon && lp_read_only(module_id) && remove_sent_files) {
++ rprintf(FERROR,
++ "ERROR: --remove-sent-files cannot be used with a read-only module\n");
++ exit_cleanup(RERR_SYNTAX);
++ return;
++ }
+
+ if (!relative_paths && !push_dir(dir)) {
+ rsyserr(FERROR, errno, "push_dir#3 %s failed",
+@@ -673,6 +681,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)
-@@ -652,6 +655,9 @@ int client_run(int f_in, int f_out, pid_
++ io_start_multiplex_in();
+
+ recv_filter_list(f_in);
+ do_server_sender(f_in, f_out, argc, argv);
+@@ -750,6 +760,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;
+ list_only |= 1;
---- orig/options.c 2004-06-20 19:30:00
-+++ options.c 2004-07-03 20:17:10
-@@ -87,6 +87,7 @@ int size_only = 0;
- int bwlimit = 0;
- size_t bwlimit_writemax = 0;
+--- orig/options.c 2005-02-16 15:35:53
++++ options.c 2005-02-15 21:33:52
+@@ -59,6 +59,7 @@ int delete_during = 0;
+ int delete_before = 0;
int delete_after = 0;
-+int delete_sent_files = 0;
+ int delete_excluded = 0;
++int remove_sent_files = 0;
+ int one_file_system = 0;
+ int protocol_version = PROTOCOL_VERSION;
+ int sparse_files = 0;
+@@ -93,6 +94,7 @@ int fuzzy_basis = 0;
+ size_t bwlimit_writemax = 0;
int only_existing = 0;
int opt_ignore_existing = 0;
- int max_delete = 0;
-@@ -94,6 +95,7 @@ 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;
-
-
-@@ -258,6 +260,7 @@ void usage(enum logcode F)
+ int max_delete = 0;
+ OFF_T max_size = 0;
+ int ignore_errors = 0;
+@@ -286,6 +288,7 @@ void usage(enum logcode F)
+ rprintf(F," --rsync-path=PATH specify path to rsync on the remote machine\n");
+ rprintf(F," --existing only update files that already exist on receiver\n");
+ rprintf(F," --ignore-existing ignore files that already exist on receiving side\n");
++ rprintf(F," --remove-sent-files sent files/symlinks are removed from sending side\n");
+ rprintf(F," --del an alias for --delete-during\n");
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");
-+ rprintf(F," --delete-sent-files updated/sent files are removed from sending side\n");
- 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");
-@@ -308,8 +311,8 @@ void usage(enum logcode F)
- }
-
- enum {OPT_VERSION = 1000, OPT_SENDER, OPT_EXCLUDE, OPT_EXCLUDE_FROM,
-- OPT_DELETE_AFTER, OPT_DELETE_EXCLUDED, OPT_LINK_DEST,
-- 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_TIMEOUT,
- OPT_REFUSED_BASE = 9000};
-
-@@ -328,6 +331,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 },
-+ {"delete-sent-files",0, POPT_ARG_NONE, 0, OPT_DELETE_SENT_FILES, 0, 0 },
+ rprintf(F," --delete-before receiver deletes before transfer (default)\n");
+@@ -369,6 +372,7 @@ static struct poptOption long_options[]
+ {"delete-during", 0, POPT_ARG_NONE, &delete_during, 0, 0, 0 },
+ {"delete-after", 0, POPT_ARG_NONE, &delete_after, 0, 0, 0 },
+ {"delete-excluded", 0, POPT_ARG_NONE, &delete_excluded, 0, 0, 0 },
++ {"remove-sent-files",0, POPT_ARG_NONE, &remove_sent_files, 0, 0, 0 },
{"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 },
-@@ -513,6 +517,11 @@ int parse_arguments(int *argc, const cha
- delete_mode = 1;
- break;
+ {"filter", 'f', POPT_ARG_STRING, 0, OPT_FILTER, 0, 0 },
+@@ -969,6 +973,17 @@ int parse_arguments(int *argc, const cha
+ return 0;
+ }
-+ case OPT_DELETE_SENT_FILES:
-+ delete_sent_files = 1;
-+ need_messages_from_generator = 1;
-+ break;
++ if (remove_sent_files) {
++ /* We only want to infer this refusal of --remove-sent-files
++ * via the refusal of "delete", not any of the "delete-FOO"
++ * options. */
++ if (refused_delete && am_sender) {
++ create_refuse_error(refused_delete);
++ return 0;
++ }
++ need_messages_from_generator = 1;
++ }
+
- case OPT_EXCLUDE:
- if (am_server || sanitize_paths)
- return 0; /* Impossible... */
-@@ -989,6 +998,9 @@ void server_options(char **args,int *arg
- }
- }
+ *argv = poptGetArgs(pc);
+ *argc = count_args(*argv);
+
+@@ -1411,6 +1426,9 @@ void server_options(char **args,int *arg
+ if (fuzzy_basis && am_sender)
+ args[ac++] = "--fuzzy";
-+ if (delete_sent_files)
-+ args[ac++] = "--delete-sent-files";
++ if (remove_sent_files)
++ args[ac++] = "--remove-sent-files";
+
*argc = ac;
return;
---- orig/receiver.c 2004-07-02 18:23:01
-+++ receiver.c 2004-07-03 20:17:10
-@@ -48,6 +48,7 @@ extern int ignore_errors;
+--- orig/receiver.c 2005-02-16 15:35:53
++++ receiver.c 2005-02-15 21:33:52
+@@ -43,6 +43,7 @@ extern int basis_dir_cnt;
+ extern int make_backups;
+ extern int do_progress;
+ extern int cleanup_got_literal;
++extern int remove_sent_files;
+ extern int module_id;
+ extern int ignore_errors;
extern int orig_umask;
- extern int keep_partial;
- extern int checksum_seed;
-+extern int delete_sent_files;
-
- static void delete_one(char *fn, int is_dir)
- {
-@@ -296,7 +297,7 @@ int recv_files(int f_in,struct file_list
+@@ -310,7 +311,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;
+ uchar *delayed_bits = NULL;
struct file_struct *file;
-@@ -478,16 +479,20 @@ int recv_files(int f_in,struct file_list
+@@ -588,7 +589,12 @@ int recv_files(int f_in, struct file_lis
cleanup_disable();
- if (!recv_ok) {
+ if (recv_ok) {
-+ if (delete_sent_files) {
++ if (remove_sent_files) {
+ SIVAL(numbuf, 0, i);
+ 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) {
+@@ -612,9 +618,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);
}
}
}
---- orig/rsync.h 2004-05-16 07:58:12
-+++ 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 */
- #define FLAG_MOUNT_POINT (1<<2) /* sender only */
-+#define FLAG_SENT (1<<7) /* sender only */
-
- /* update this if you make incompatible changes */
- #define PROTOCOL_VERSION 28
-@@ -123,6 +124,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 {
-+ MSG_SUCCESS=6, /* successfully updated indicated flist index */
- 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 */
---- orig/rsync.yo 2004-06-17 06:32:00
-+++ rsync.yo 2004-07-03 20:17:10
-@@ -313,6 +313,7 @@ verb(
+--- orig/rsync.yo 2005-02-16 15:35:54
++++ rsync.yo 2005-02-15 21:33:55
+@@ -332,6 +332,7 @@ to the detailed description below for a
+ --rsync-path=PATH specify path to rsync on the remote machine
+ --existing only update files that already exist
+ --ignore-existing ignore files that already exist on receiver
++ --remove-sent-files sent files/symlinks are removed from sender
+ --del an alias for --delete-during
--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
-+ --delete-sent-files updated/sent files are removed from sender
- --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
-@@ -603,6 +604,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.
+ --delete-before receiver deletes before transfer (default)
+@@ -665,6 +666,11 @@ dit(bf(--ignore-existing))
+ This tells rsync not to update files that already exist on
+ the destination.
-+dit(bf(--delete-sent-files)) This tells rsync to remove the source files
-+on the sending side that are successfully transferred to the receiving
-+side. Directories are not removed, nor are files that are identical on
-+both systems.
++dit(bf(--remove-sent-files)) This tells rsync to remove from the sending
++side the files and/or symlinks that are newly created or whose content is
++updated on the receiving side. Directories and devices are not removed,
++nor are files/symlinks whose attributes are merely changed.
+
- dit(bf(--ignore-errors)) Tells --delete to go ahead and delete files
- even when there are I/O errors.
+ dit(bf(--delete)) This tells rsync to delete extraneous files from the
+ receiving side (ones that aren't on the sending side), but only for the
+ directories that are being synchronized. You must have asked rsync to
+--- orig/rsyncd.conf.yo 2005-02-16 15:35:54
++++ rsyncd.conf.yo 2005-02-15 21:33:55
+@@ -443,6 +443,10 @@ quote(tt( refuse options = c delete))
+
+ The reason the above refuses all delete options is that the options imply
+ bf(--delete), and implied options are refused just like explicit options.
++As an additional safety feature, the refusal of "delete" also refuses
++bf(remove-sent-files) when the daemon is the sender; if you want the latter
++without the former, instead refuse "delete-*" -- that refuses all the
++delete modes without affecting bf(--remove-sent-files).
---- orig/sender.c 2004-06-18 16:55:00
-+++ sender.c 2004-07-03 20:17:10
-@@ -27,6 +27,7 @@ extern int dry_run;
+ When an option is refused, the server prints an error message and exits.
+ To prevent all compression, you can use "dont compress = *" (see below)
+--- orig/sender.c 2005-02-16 15:35:54
++++ sender.c 2005-02-15 21:33:55
+@@ -28,6 +28,7 @@ extern int io_error;
+ extern int dry_run;
extern int am_server;
extern int am_daemon;
++extern int remove_sent_files;
extern int protocol_version;
-+extern int delete_sent_files;
-
-
- /**
-@@ -104,7 +105,29 @@ static struct sum_struct *receive_sums(i
+ extern int updating_basis_file;
+ extern int make_backups;
+@@ -98,7 +99,32 @@ static struct sum_struct *receive_sums(i
return s;
}
+ struct file_struct *file;
+ unsigned int offset;
+
-+ if (!the_flist)
++ if (!the_flist || i < 0 || i >= the_flist->count)
+ return;
+
+ file = the_flist->files[i];
-+ if (!(file->flags & FLAG_SENT))
-+ return; /* We didn't send it -- impossible! */
-+ if (file->basedir) {
++ /* The generator might tell us about symlinks we didn't send. */
++ if (!(file->flags & FLAG_SENT) && !S_ISLNK(file->mode))
++ return;
++ if (file->dir.root) {
+ offset = stringjoin(fname, sizeof fname,
-+ file->basedir, "/", NULL);
++ file->dir.root, "/", NULL);
+ } else
+ offset = 0;
+ f_name_to(file, fname + offset);
-+ if (delete_sent_files && do_unlink(fname) == 0 && verbose > 0)
-+ rprintf(FINFO, "sender removed %s\n", fname + offset);
++ if (remove_sent_files && do_unlink(fname) == 0 && verbose) {
++ rprintf(FINFO, "sender removed %s\n",
++ safe_fname(fname + offset));
++ }
+}
void send_files(struct file_list *flist, int f_out, int f_in)
{
-@@ -129,6 +152,8 @@ void send_files(struct file_list *flist,
+@@ -117,6 +143,8 @@ void send_files(struct file_list *flist,
if (verbose > 2)
rprintf(FINFO, "send_files starting\n");
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);
+@@ -281,6 +309,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)