X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/70891d2683929346847f5a7eccc135ee0352c4ec..f2acdf8c9d3000cb9f69f2a66166e407c28e5a50:/flags.diff diff --git a/flags.diff b/flags.diff index 4a008d0..69072ef 100644 --- a/flags.diff +++ b/flags.diff @@ -10,29 +10,57 @@ To use this patch, run these commands for a successful build: TODO: fix --delete-delay to work with --flags option. +--- old/compat.c ++++ new/compat.c +@@ -45,6 +45,7 @@ extern int prune_empty_dirs; + extern int protocol_version; + extern int preserve_uid; + extern int preserve_gid; ++extern int preserve_fileflags; + extern int preserve_acls; + extern int preserve_xattrs; + extern int need_messages_from_generator; +@@ -58,7 +59,7 @@ extern char *dest_option; + extern struct filter_list_struct filter_list; + + /* These index values are for the file-list's extra-attribute array. */ +-int uid_ndx, gid_ndx, acls_ndx, xattrs_ndx; ++int uid_ndx, gid_ndx, fileflags_ndx, acls_ndx, xattrs_ndx; + + /* The server makes sure that if either side only supports a pre-release + * version of a protocol, that both sides must speak a compatible version +@@ -105,6 +106,8 @@ void setup_protocol(int f_out,int f_in) + uid_ndx = ++file_extra_cnt; + if (preserve_gid) + gid_ndx = ++file_extra_cnt; ++ if (preserve_fileflags) ++ fileflags_ndx = ++file_extra_cnt; + if (preserve_acls && !am_sender) + acls_ndx = ++file_extra_cnt; + if (preserve_xattrs) --- old/configure.in +++ new/configure.in -@@ -549,7 +549,7 @@ AC_CHECK_FUNCS(waitpid wait4 getcwd strd +@@ -560,7 +560,7 @@ AC_CHECK_FUNCS(waitpid wait4 getcwd strd memmove lchown vsnprintf snprintf vasprintf asprintf setsid glob strpbrk \ strlcat strlcpy strtol mallinfo getgroups setgroups geteuid getegid \ setlocale setmode open64 lseek64 mkstemp64 mtrace va_copy __va_copy \ -- strerror putenv iconv_open locale_charset nl_langinfo \ -+ chflags strerror putenv iconv_open locale_charset nl_langinfo \ - sigaction sigprocmask) +- strerror putenv iconv_open locale_charset nl_langinfo getxattr \ ++ chflags strerror putenv iconv_open locale_charset nl_langinfo getxattr \ + extattr_get_link sigaction sigprocmask) AC_CHECK_FUNCS(getpgrp tcgetpgrp) --- old/flist.c +++ new/flist.c -@@ -44,6 +44,7 @@ extern int preserve_links; +@@ -50,6 +50,7 @@ extern int preserve_links; extern int preserve_hard_links; extern int preserve_devices; extern int preserve_specials; -+extern int preserve_fileflags; - extern int preserve_uid; - extern int preserve_gid; - extern int relative_paths; -@@ -313,6 +314,9 @@ static void send_file_entry(struct file_ - unsigned short flags; ++extern int fileflags_ndx; + extern int uid_ndx; + extern int gid_ndx; + extern int eol_nulls; +@@ -362,6 +363,9 @@ static void send_file_entry(int f, struc + { static time_t modtime; static mode_t mode; +#ifdef SUPPORT_FLAGS @@ -41,33 +69,33 @@ TODO: fix --delete-delay to work with --flags option. static int64 dev; static dev_t rdev; static uint32 rdev_major; -@@ -343,6 +347,12 @@ static void send_file_entry(struct file_ - flags |= XMIT_SAME_MODE; +@@ -410,6 +414,12 @@ static void send_file_entry(int f, struc + xflags |= XMIT_SAME_MODE; else mode = file->mode; +#ifdef SUPPORT_FLAGS -+ if (file->fileflags == fileflags) -+ flags |= XMIT_SAME_FLAGS; ++ if (F_FFLAGS(file) == fileflags) ++ xflags |= XMIT_SAME_FLAGS; + else -+ fileflags = file->fileflags; ++ fileflags = F_FFLAGS(file); +#endif - if ((preserve_devices && IS_DEVICE(mode)) - || (preserve_specials && IS_SPECIAL(mode))) { - if (protocol_version < 28) { -@@ -429,6 +439,10 @@ static void send_file_entry(struct file_ - write_int(f, modtime); - if (!(flags & XMIT_SAME_MODE)) + + if (protocol_version >= 30 && S_ISDIR(mode) && !(file->flags & FLAG_XFER_DIR)) + xflags |= XMIT_NON_XFER_DIR; +@@ -532,6 +542,10 @@ static void send_file_entry(int f, struc + } + if (!(xflags & XMIT_SAME_MODE)) write_int(f, to_wire_mode(mode)); +#ifdef SUPPORT_FLAGS -+ if (preserve_fileflags && !(flags & XMIT_SAME_FLAGS)) ++ if (fileflags_ndx && !(xflags & XMIT_SAME_FLAGS)) + write_int(f, (int)fileflags); +#endif - if (preserve_uid && !(flags & XMIT_SAME_UID)) { - if (!numeric_ids) - add_uid(uid); -@@ -497,6 +511,9 @@ static struct file_struct *recv_file_ent + if (uid_ndx && !(xflags & XMIT_SAME_UID)) { + if (protocol_version < 30) + write_int(f, uid); +@@ -620,6 +634,9 @@ static struct file_struct *recv_file_ent { - static time_t modtime; + static int64 modtime; static mode_t mode; +#ifdef SUPPORT_FLAGS + static uint32 fileflags; @@ -75,58 +103,60 @@ TODO: fix --delete-delay to work with --flags option. static int64 dev; static dev_t rdev; static uint32 rdev_major; -@@ -571,9 +588,12 @@ static struct file_struct *recv_file_ent - modtime = (time_t)read_int(f); - if (!(flags & XMIT_SAME_MODE)) +@@ -755,9 +772,12 @@ static struct file_struct *recv_file_ent + } + if (!(xflags & XMIT_SAME_MODE)) mode = from_wire_mode(read_int(f)); - if (chmod_modes && !S_ISLNK(mode)) mode = tweak_mode(mode, chmod_modes); +#ifdef SUPPORT_FLAGS -+ if (preserve_fileflags && !(flags & XMIT_SAME_FLAGS)) ++ if (fileflags_ndx && !(xflags & XMIT_SAME_FLAGS)) + fileflags = (uint32)read_int(f); +#endif - if (preserve_uid && !(flags & XMIT_SAME_UID)) - uid = (uid_t)read_int(f); -@@ -639,6 +659,10 @@ static struct file_struct *recv_file_ent - file->modtime = modtime; - file->length = file_length; + if (uid_ndx && !(xflags & XMIT_SAME_UID)) { + if (protocol_version < 30) +@@ -877,6 +897,10 @@ static struct file_struct *recv_file_ent + OPT_EXTRA(file, 0)->unum = (uint32)(file_length >> 32); + } file->mode = mode; +#ifdef SUPPORT_FLAGS -+ if (preserve_fileflags) ++ if (fileflags_ndx) + F_FFLAGS(file) = fileflags; +#endif - if (preserve_uid) - F_UID(file) = uid; - if (preserve_gid) -@@ -902,6 +926,10 @@ struct file_struct *make_file(const char - file->modtime = st.st_mtime; - file->length = st.st_size; + if (uid_ndx) + F_OWNER(file) = uid; + if (gid_ndx) { +@@ -1194,6 +1218,10 @@ struct file_struct *make_file(const char + OPT_EXTRA(file, 0)->unum = (uint32)(st.st_size >> 32); + } file->mode = st.st_mode; +#ifdef SUPPORT_FLAGS -+ if (preserve_fileflags) ++ if (fileflags_ndx) + F_FFLAGS(file) = st.st_flags; +#endif - if (preserve_uid) - F_UID(file) = st.st_uid; - if (preserve_gid) + if (uid_ndx) + F_OWNER(file) = st.st_uid; + if (gid_ndx) --- old/generator.c +++ new/generator.c -@@ -101,6 +101,12 @@ int non_perishable_cnt = 0; - static int deletion_count = 0; /* used to implement --max-delete */ - static FILE *delete_delay_fp = NULL; +@@ -115,6 +115,14 @@ static int need_retouch_dir_times; + static int need_retouch_dir_perms; + static const char *solo_file = NULL; +#ifdef SUPPORT_FLAGS -+#define FILEFLAGS(ff) ff ++#define FF_PTR(p) F_FFLAGS(p) ++#define FF_STAT(s) s.st_flags +#else -+#define FILEFLAGS(ff) 0 ++#define FF_PTR(p) 0 ++#define FF_STAT(s) 0 +#endif + /* For calling delete_item() and delete_dir_contents(). */ #define DEL_RECURSE (1<<1) /* recurse */ #define DEL_DIR_IS_EMPTY (1<<2) /* internal delete_FUNCTIONS use only */ -@@ -116,7 +122,6 @@ enum delret { +@@ -130,7 +138,6 @@ enum delret { /* Forward declaration for delete_item(). */ static enum delret delete_dir_contents(char *fname, int flags); @@ -134,7 +164,7 @@ TODO: fix --delete-delay to work with --flags option. static int is_backup_file(char *fn) { int k = strlen(fn) - backup_suffix_len; -@@ -129,17 +134,20 @@ static int is_backup_file(char *fn) +@@ -143,17 +150,20 @@ static int is_backup_file(char *fn) * Note that fbuf must point to a MAXPATHLEN buffer if the mode indicates it's * a directory! (The buffer is used for recursion, but returned unchanged.) */ @@ -158,122 +188,166 @@ TODO: fix --delete-delay to work with --flags option. if (S_ISDIR(mode) && !(flags & DEL_DIR_IS_EMPTY)) { ignore_perishable = 1; /* If DEL_RECURSE is not set, this just reports emptiness. */ -@@ -251,7 +259,7 @@ static enum delret delete_dir_contents(c +@@ -265,7 +275,7 @@ 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; - if (delete_item(fname, fp->mode, NULL, flags) != DR_SUCCESS) -+ if (delete_item(fname, fp->mode, FILEFLAGS(fp->fileflags), NULL, flags) != DR_SUCCESS) ++ if (delete_item(fname, fp->mode, FF_PTR(fp), NULL, flags) != DR_SUCCESS) ret = DR_NOT_EMPTY; } -@@ -403,10 +411,10 @@ static void delete_in_dir(struct file_li - } - if (flist_find(flist, fp) < 0) { - f_name(fp, delbuf); -- if (delete_delay_fp) -+ if (delete_delay_fp) /* XXX need to output fileflags value here too */ - fprintf(delete_delay_fp, "%o %s%c", (short)fp->mode, delbuf, '\0'); - else +@@ -321,8 +331,9 @@ static int remember_delete(struct file_s + + while (1) { + len = snprintf(deldelay_buf + deldelay_cnt, +- deldelay_size - deldelay_cnt, +- "%x %s%c", (int)file->mode, fname, '\0'); ++ deldelay_size - deldelay_cnt, "%x %x %s%c", ++ (int)file->mode, (int)FF_PTR(file), ++ fname, '\0'); + if ((deldelay_cnt += len) <= deldelay_size) + break; + if (deldelay_fd < 0 && !start_delete_delay_temp()) +@@ -335,7 +346,7 @@ static int remember_delete(struct file_s + return 1; + } + +-static int read_delay_line(char *buf) ++static int read_delay_line(char *buf, int *fileflags_p) + { + static int read_pos = 0; + int j, len, mode; +@@ -377,7 +388,7 @@ static int read_delay_line(char *buf) + + bp = deldelay_buf + read_pos; + +- if (sscanf(bp, "%x ", &mode) != 1) { ++ if (sscanf(bp, "%x %x ", &mode, fileflags_p) != 2) { + invalid_data: + rprintf(FERROR, "ERROR: invalid data in delete-delay file.\n"); + return -1; +@@ -400,15 +411,15 @@ static int read_delay_line(char *buf) + + static void do_delayed_deletions(char *delbuf) + { +- int mode; ++ int mode, fileflags; + + if (deldelay_fd >= 0) { + if (deldelay_cnt && !flush_delete_delay()) + return; + lseek(deldelay_fd, 0, 0); + } +- while ((mode = read_delay_line(delbuf)) >= 0) +- delete_item(delbuf, mode, NULL, DEL_RECURSE); ++ while ((mode = read_delay_line(delbuf, &fileflags)) >= 0) ++ delete_item(delbuf, mode, fileflags, NULL, DEL_RECURSE); + if (deldelay_fd >= 0) + close(deldelay_fd); + } +@@ -475,7 +486,7 @@ static void delete_in_dir(char *fbuf, st + if (!remember_delete(fp, delbuf)) + break; + } else - delete_item(delbuf, fp->mode, NULL, DEL_RECURSE); -+ delete_item(delbuf, fp->mode, FILEFLAGS(fp->fileflags), NULL, DEL_RECURSE); ++ delete_item(delbuf, fp->mode, FF_PTR(fp), NULL, DEL_RECURSE); } } -@@ -1101,7 +1109,7 @@ static void recv_generator(char *fname, +@@ -1219,7 +1230,7 @@ static void recv_generator(char *fname, * we need to delete it. If it doesn't exist, then * (perhaps recursively) create it. */ - if (statret == 0 && !S_ISDIR(st.st_mode)) { -- if (delete_item(fname, st.st_mode, "directory", del_opts) != 0) -+ if (delete_item(fname, st.st_mode, FILEFLAGS(st.st_flags), "directory", del_opts) != 0) - return; + if (statret == 0 && !S_ISDIR(sx.st.st_mode)) { +- if (delete_item(fname, sx.st.st_mode, "directory", del_opts) != 0) ++ if (delete_item(fname, sx.st.st_mode, FF_STAT(sx.st), "directory", del_opts) != 0) + goto skipping_dir_contents; statret = -1; } -@@ -1195,7 +1203,7 @@ static void recv_generator(char *fname, +@@ -1351,7 +1362,7 @@ static void recv_generator(char *fname, } /* Not the right symlink (or not a symlink), so * delete it. */ -- if (delete_item(fname, st.st_mode, "symlink", del_opts) != 0) -+ if (delete_item(fname, st.st_mode, FILEFLAGS(st.st_flags), "symlink", del_opts) != 0) - return; +- if (delete_item(fname, sx.st.st_mode, "symlink", del_opts) != 0) ++ if (delete_item(fname, sx.st.st_mode, FF_STAT(sx.st), "symlink", del_opts) != 0) + goto cleanup; } else if (basis_dir[0] != NULL) { - int j = try_dests_non(file, fname, ndx, fnamecmpbuf, &st, -@@ -1267,7 +1275,7 @@ static void recv_generator(char *fname, + int j = try_dests_non(file, fname, ndx, fnamecmpbuf, &sx, +@@ -1430,7 +1441,7 @@ static void recv_generator(char *fname, goto return_with_success; - return; + goto cleanup; } -- if (delete_item(fname, st.st_mode, t, del_opts) != 0) -+ if (delete_item(fname, st.st_mode, FILEFLAGS(st.st_flags), t, del_opts) != 0) - return; +- if (delete_item(fname, sx.st.st_mode, t, del_opts) != 0) ++ if (delete_item(fname, sx.st.st_mode, FF_STAT(sx.st), t, del_opts) != 0) + goto cleanup; } else if (basis_dir[0] != NULL) { - int j = try_dests_non(file, fname, ndx, fnamecmpbuf, &st, -@@ -1353,7 +1361,7 @@ static void recv_generator(char *fname, + int j = try_dests_non(file, fname, ndx, fnamecmpbuf, &sx, +@@ -1521,7 +1532,7 @@ static void recv_generator(char *fname, fnamecmp_type = FNAMECMP_FNAME; - if (statret == 0 && !S_ISREG(st.st_mode)) { -- if (delete_item(fname, st.st_mode, "regular file", del_opts) != 0) -+ if (delete_item(fname, st.st_mode, FILEFLAGS(st.st_flags), "regular file", del_opts) != 0) - return; + if (statret == 0 && !S_ISREG(sx.st.st_mode)) { +- if (delete_item(fname, sx.st.st_mode, "regular file", del_opts) != 0) ++ if (delete_item(fname, sx.st.st_mode, FF_STAT(sx.st), "regular file", del_opts) != 0) + goto cleanup; statret = -1; stat_errno = ENOENT; --- old/options.c +++ new/options.c -@@ -48,6 +48,7 @@ int copy_links = 0; - int preserve_links = 0; - int preserve_hard_links = 0; +@@ -50,6 +50,7 @@ int preserve_hard_links = 0; + int preserve_acls = 0; + int preserve_xattrs = 0; int preserve_perms = 0; +int preserve_fileflags = 0; int preserve_executability = 0; int preserve_devices = 0; int preserve_specials = 0; -@@ -201,6 +202,7 @@ static void print_rsync_version(enum log - char const *hardlinks = "no "; +@@ -220,6 +221,7 @@ static void print_rsync_version(enum log char const *links = "no "; + char const *iconv = "no "; char const *ipv6 = "no "; + char const *fileflags = "no "; STRUCT_STAT *dumstat; - #ifdef HAVE_SOCKETPAIR -@@ -223,6 +225,10 @@ static void print_rsync_version(enum log - ipv6 = ""; + #if SUBPROTOCOL_VERSION != 0 +@@ -249,6 +251,9 @@ static void print_rsync_version(enum log + #ifdef ICONV_OPTION + iconv = ""; #endif - +#ifdef SUPPORT_FLAGS + fileflags = ""; +#endif -+ - rprintf(f, "%s version %s protocol version %d\n", - RSYNC_NAME, RSYNC_VERSION, PROTOCOL_VERSION); - rprintf(f, "Copyright (C) 1996-2006 by Andrew Tridgell, Wayne Davison, and others.\n"); -@@ -233,8 +239,8 @@ static void print_rsync_version(enum log + + rprintf(f, "%s version %s protocol version %d%s\n", + RSYNC_NAME, RSYNC_VERSION, PROTOCOL_VERSION, subprotocol); +@@ -262,8 +267,8 @@ static void print_rsync_version(enum log (int)(sizeof (int64) * 8)); rprintf(f, " %ssocketpairs, %shardlinks, %ssymlinks, %sIPv6, batchfiles, %sinplace,\n", got_socketpair, hardlinks, links, ipv6, have_inplace); -- rprintf(f, " %sappend\n", -- have_inplace); -+ rprintf(f, " %sappend, %sfile-flags\n", -+ have_inplace, fileflags); +- rprintf(f, " %sappend, %sACLs, %sxattrs, %siconv\n", +- have_inplace, acls, xattrs, iconv); ++ rprintf(f, " %sappend, %sACLs, %sxattrs, %siconv, %sfile-flags\n", ++ have_inplace, acls, xattrs, iconv, fileflags); #ifdef MAINTAINER_MODE rprintf(f, "Panic Action: \"%s\"\n", get_panic_action()); -@@ -300,6 +306,7 @@ void usage(enum logcode F) +@@ -330,6 +335,7 @@ void usage(enum logcode F) rprintf(F," -K, --keep-dirlinks treat symlinked dir on receiver as dir\n"); rprintf(F," -H, --hard-links preserve hard links\n"); rprintf(F," -p, --perms preserve permissions\n"); + rprintf(F," --fileflags preserve file-flags\n"); rprintf(F," -E, --executability preserve the file's executability\n"); rprintf(F," --chmod=CHMOD affect file and/or directory permissions\n"); - rprintf(F," -o, --owner preserve owner (super-user only)\n"); -@@ -421,6 +428,8 @@ static struct poptOption long_options[] + #ifdef SUPPORT_ACLS +@@ -469,6 +475,8 @@ static struct poptOption long_options[] {"perms", 'p', POPT_ARG_VAL, &preserve_perms, 1, 0, 0 }, {"no-perms", 0, POPT_ARG_VAL, &preserve_perms, 0, 0, 0 }, {"no-p", 0, POPT_ARG_VAL, &preserve_perms, 0, 0, 0 }, + {"fileflags", 0, POPT_ARG_VAL, &preserve_fileflags, 1, 0, 0 }, + {"no-fileflags", 0, POPT_ARG_VAL, &preserve_fileflags, 0, 0, 0 }, {"executability", 'E', POPT_ARG_NONE, &preserve_executability, 0, 0, 0 }, - {"times", 't', POPT_ARG_VAL, &preserve_times, 1, 0, 0 }, - {"no-times", 0, POPT_ARG_VAL, &preserve_times, 0, 0, 0 }, -@@ -1126,6 +1135,15 @@ int parse_arguments(int *argc, const cha + {"acls", 'A', POPT_ARG_NONE, 0, 'A', 0, 0 }, + {"no-acls", 0, POPT_ARG_VAL, &preserve_acls, 0, 0, 0 }, +@@ -1279,6 +1287,15 @@ int parse_arguments(int *argc_p, const c } #endif @@ -289,16 +363,7 @@ TODO: fix --delete-delay to work with --flags option. if (write_batch && read_batch) { snprintf(err_buf, sizeof err_buf, "--write-batch and --read-batch can not be used together\n"); -@@ -1223,6 +1241,8 @@ int parse_arguments(int *argc, const cha - preserve_uid = flist_extra_ndx++; - if (preserve_gid) - preserve_gid = flist_extra_ndx++; -+ if (preserve_fileflags) -+ preserve_fileflags = flist_extra_ndx++; - - *argv = poptGetArgs(pc); - *argc = count_args(*argv); -@@ -1585,6 +1605,9 @@ void server_options(char **args,int *arg +@@ -1770,6 +1787,9 @@ void server_options(char **args, int *ar if (xfer_dirs && !recurse && delete_mode && am_sender) args[ac++] = "--no-r"; @@ -310,20 +375,32 @@ TODO: fix --delete-delay to work with --flags option. goto oom; --- old/rsync.c +++ new/rsync.c -@@ -30,9 +30,12 @@ - #include - #endif - -+#define NOCHANGE_FLAGS (UF_IMMUTABLE|UF_APPEND|UF_NOUNLINK|SF_IMMUTABLE|SF_APPEND|SF_NOUNLINK) -+ - extern int verbose; - extern int dry_run; +@@ -31,6 +31,7 @@ extern int dry_run; + extern int preserve_acls; + extern int preserve_xattrs; extern int preserve_perms; +extern int preserve_fileflags; extern int preserve_executability; extern int preserve_times; - extern int omit_dir_times; -@@ -176,6 +179,41 @@ mode_t dest_mode(mode_t flist_mode, mode + extern int am_root; +@@ -60,6 +61,16 @@ iconv_t ic_send = (iconv_t)-1, ic_recv = + int ic_ndx; + #endif + ++#ifdef SUPPORT_FLAGS ++#ifndef UF_NOUNLINK ++#define UF_NOUNLINK 0 ++#endif ++#ifndef SF_NOUNLINK ++#define SF_NOUNLINK 0 ++#endif ++#define NOCHANGE_FLAGS (UF_IMMUTABLE|UF_APPEND|UF_NOUNLINK|SF_IMMUTABLE|SF_APPEND|SF_NOUNLINK) ++#endif ++ + static const char *default_charset(void) + { + #if defined HAVE_LIBCHARSET_H && defined HAVE_LOCALE_CHARSET +@@ -267,6 +278,41 @@ mode_t dest_mode(mode_t flist_mode, mode return new_mode; } @@ -362,17 +439,17 @@ TODO: fix --delete-delay to work with --flags option. +} +#endif + - int set_file_attrs(char *fname, struct file_struct *file, STRUCT_STAT *st, - int flags) + int set_file_attrs(const char *fname, struct file_struct *file, statx *sxp, + const char *fnamecmp, int flags) { -@@ -274,6 +312,15 @@ int set_file_attrs(char *fname, struct f +@@ -395,6 +441,15 @@ int set_file_attrs(const char *fname, st } #endif +#ifdef SUPPORT_FLAGS -+ if (preserve_fileflags && !S_ISLNK(st->st_mode) -+ && st->st_flags != file->fileflags) { -+ if (!set_fileflags(fname, file->fileflags)) ++ if (preserve_fileflags && !S_ISLNK(sxp->st.st_mode) ++ && sxp->st.st_flags != F_FFLAGS(file)) { ++ if (!set_fileflags(fname, F_FFLAGS(file))) + return 0; + updated = 1; + } @@ -381,38 +458,38 @@ TODO: fix --delete-delay to work with --flags option. if (verbose > 1 && flags & ATTRS_REPORT) { if (updated) rprintf(FCLIENT, "%s\n", fname); -@@ -321,6 +368,9 @@ void finish_transfer(char *fname, char * - set_file_attrs(fnametmp, file, NULL, +@@ -454,6 +509,9 @@ void finish_transfer(const char *fname, + set_file_attrs(fnametmp, file, NULL, fnamecmp, ok_to_set_time ? 0 : ATTRS_SKIP_MTIME); +#ifdef SUPPORT_FLAGS -+ make_mutable(fnametmp, file->mode, file->fileflags); ++ make_mutable(fnametmp, file->mode, F_FFLAGS(file)); +#endif /* move tmp file over real file */ if (verbose > 2) rprintf(FINFO, "renaming %s to %s\n", fnametmp, fname); -@@ -335,6 +385,9 @@ void finish_transfer(char *fname, char * +@@ -468,6 +526,9 @@ void finish_transfer(const char *fname, } if (ret == 0) { /* The file was moved into place (not copied), so it's done. */ +#ifdef SUPPORT_FLAGS -+ undo_make_mutable(fname, file->mode, file->fileflags); ++ undo_make_mutable(fname, file->mode, F_FFLAGS(file)); +#endif return; } /* The file was copied, so tweak the perms of the copied file. If it --- old/rsync.h +++ new/rsync.h -@@ -54,6 +54,7 @@ - #define XMIT_HAS_IDEV_DATA (1<<9) - #define XMIT_SAME_DEV (1<<10) - #define XMIT_RDEV_MINOR_IS_SMALL (1<<11) -+#define XMIT_SAME_FLAGS (1<<12) +@@ -57,6 +57,7 @@ + #define XMIT_RDEV_MINOR_8_pre30 (1<<11) /* protocols 28 - 29 */ + #define XMIT_GROUP_NAME_FOLLOWS (1<<11) /* protocols 30 - now */ + #define XMIT_HLINK_FIRST (1<<12) /* protocols 30 - now (HLINKED files only) */ ++#define XMIT_SAME_FLAGS (1<<14) /* protocols ?? - now */ /* These flags are used in the live flist data. */ -@@ -347,6 +348,10 @@ enum msgcode { - #define schar char +@@ -406,6 +407,10 @@ enum msgcode { + #endif #endif +#ifdef HAVE_CHFLAGS @@ -422,25 +499,33 @@ TODO: fix --delete-delay to work with --flags option. /* Find a variable that is either exactly 32-bits or longer. * If some code depends on 32-bit truncation, it will need to * take special action in a "#if SIZEOF_INT32 > 4" section. */ -@@ -540,6 +545,7 @@ union flist_extras { - /* When enabled, all entries have these: */ - #define F_UID(f) FLIST_EXTRA(f, preserve_uid).uid - #define F_GID(f) FLIST_EXTRA(f, preserve_gid).gid -+#define F_FFLAGS(f) FLIST_EXTRA(f, preserve_fileflags).num - - /* These are per-entry optional and mutally exclusive: */ - #define F_IDEV(f) FLIST_EXTRA(f, flist_extra_ndx).idev +@@ -573,6 +578,7 @@ extern int file_extra_cnt; + extern int inc_recurse; + extern int uid_ndx; + extern int gid_ndx; ++extern int fileflags_ndx; + extern int acls_ndx; + extern int xattrs_ndx; + +@@ -610,6 +616,7 @@ extern int xattrs_ndx; + /* When the associated option is on, all entries will have these present: */ + #define F_OWNER(f) REQ_EXTRA(f, uid_ndx)->unum + #define F_GROUP(f) REQ_EXTRA(f, gid_ndx)->unum ++#define F_FFLAGS(f) REQ_EXTRA(f, fileflags_ndx)->unum + #define F_ACL(f) REQ_EXTRA(f, acls_ndx)->num + #define F_XATTR(f) REQ_EXTRA(f, xattrs_ndx)->num + #define F_NDX(f) REQ_EXTRA(f, ic_ndx)->num --- old/rsync.yo +++ new/rsync.yo -@@ -321,6 +321,7 @@ to the detailed description below for a +@@ -343,6 +343,7 @@ to the detailed description below for a -K, --keep-dirlinks treat symlinked dir on receiver as dir -H, --hard-links preserve hard links -p, --perms preserve permissions + --flags preserve file flags -E, --executability preserve executability --chmod=CHMOD affect file and/or directory permissions - -o, --owner preserve owner (super-user only) -@@ -510,7 +511,9 @@ specified, in which case bf(-r) is not i + -A, --acls preserve ACLs (implies -p) +@@ -543,7 +544,9 @@ specified, in which case bf(-r) is not i Note that bf(-a) bf(does not preserve hardlinks), because finding multiply-linked files is expensive. You must separately @@ -451,9 +536,9 @@ TODO: fix --delete-delay to work with --flags option. dit(--no-OPTION) You may turn off one or more implied options by prefixing the option name with "no-". Not all options may be prefixed with a "no-": -@@ -805,6 +808,13 @@ quote(itemization( - - If bf(--perms) is enabled, this option is ignored. +@@ -883,6 +886,13 @@ dit(bf(-X, --xattrs)) This option causes + extended attributes to be the same as the local ones. This will work + only if the remote machine's rsync also supports this option. +dit(bf(--flags)) This option causes rsync to update the change file flags +to be the same as the source file, if your OS supports the bf(chflags)(2) @@ -467,7 +552,7 @@ TODO: fix --delete-delay to work with --flags option. transfer. The resulting value is treated as though it was the permissions --- old/syscall.c +++ new/syscall.c -@@ -153,6 +153,15 @@ int do_chmod(const char *path, mode_t mo +@@ -162,6 +162,15 @@ int do_chmod(const char *path, mode_t mo } #endif