Fixed a failing hunk.
[rsync-patches.git] / sender-receiver-excludes.diff
1 This patch adds two modifiers to the filter rule prefixes that allow a
2 rule to be marked as sender-side (s), receiver-side (r), or both ("sr"
3 or omitted).  Sender-side rules prevent files from being transferred,
4 while receiver-side rules prevent files from being deleted.  The default
5 for an unmodified include/exclude rule is to affect both sides, but a
6 rule that is explicitly marked as affecting both sides will remain
7 unaffected by the --delete-excluded option (that option changes any
8 unmodified rules into server-side only rules).
9
10 See the updated manpage for the details.
11
12 --- orig/exclude.c      2005-02-11 10:53:14
13 +++ exclude.c   2005-02-12 19:08:30
14 @@ -53,7 +53,8 @@ struct filter_list_struct server_filter_
15  #define MAX_RULE_PREFIX (16)
16  
17  #define MODIFIERS_MERGE_FILE "-+Cenw"
18 -#define MODIFIERS_INCL_EXCL "/!C"
19 +#define MODIFIERS_INCL_EXCL "/!Crs"
20 +#define MODIFIERS_HIDE_PROTECT "/!"
21  
22  /* The dirbuf is set by push_local_filters() to the current subdirectory
23   * relative to curr_dir that is being processed.  The path always has a
24 @@ -678,6 +679,10 @@ static const char *parse_rule_tok(const 
25                         if ((s = RULE_STRCMP(s, "exclude")) != NULL)
26                                 ch = '-';
27                         break;
28 +               case 'h':
29 +                       if ((s = RULE_STRCMP(s, "hide")) != NULL)
30 +                               ch = 'H';
31 +                       break;
32                 case 'i':
33                         if ((s = RULE_STRCMP(s, "include")) != NULL)
34                                 ch = '+';
35 @@ -686,6 +691,19 @@ static const char *parse_rule_tok(const 
36                         if ((s = RULE_STRCMP(s, "merge")) != NULL)
37                                 ch = '.';
38                         break;
39 +               case 'p':
40 +                       if ((s = RULE_STRCMP(s, "protect")) != NULL)
41 +                               ch = 'P';
42 +                       break;
43 +               case 'r':
44 +                       if ((s = RULE_STRCMP(s, "risk")) != NULL)
45 +                               ch = 'R';
46 +                       break;
47 +               case 's':
48 +                       if ((s = RULE_STRCMP(s, "show")) != NULL)
49 +                               ch = 'S';
50 +                       break;
51 +
52                 default:
53                         ch = *s;
54                         if (s[1] == ',')
55 @@ -707,6 +725,20 @@ static const char *parse_rule_tok(const 
56                 case '-':
57                         mods = MODIFIERS_INCL_EXCL;
58                         break;
59 +               case 'S':
60 +                       new_mflags |= MATCHFLG_INCLUDE;
61 +                       /* FALL THROUGH */
62 +               case 'H':
63 +                       new_mflags |= MATCHFLG_SENDER_SIDE;
64 +                       mods = MODIFIERS_HIDE_PROTECT;
65 +                       break;
66 +               case 'R':
67 +                       new_mflags |= MATCHFLG_INCLUDE;
68 +                       /* FALL THROUGH */
69 +               case 'P':
70 +                       new_mflags |= MATCHFLG_RECEIVER_SIDE;
71 +                       mods = MODIFIERS_HIDE_PROTECT;
72 +                       break;
73                 case '!':
74                         new_mflags |= MATCHFLG_CLEAR_LIST;
75                         mods = NULL;
76 @@ -759,6 +791,12 @@ static const char *parse_rule_tok(const 
77                         case 'n':
78                                 new_mflags |= MATCHFLG_NO_INHERIT;
79                                 break;
80 +                       case 'r':
81 +                               new_mflags |= MATCHFLG_RECEIVER_SIDE;
82 +                               break;
83 +                       case 's':
84 +                               new_mflags |= MATCHFLG_SENDER_SIDE;
85 +                               break;
86                         case 'w':
87                                 new_mflags |= MATCHFLG_WORD_SPLIT;
88                                 break;
89 @@ -1013,6 +1051,13 @@ char *get_rule_prefix(int match_flags, c
90         }
91         if (match_flags & MATCHFLG_EXCLUDE_SELF)
92                 *op++ = 'e';
93 +       if (match_flags & MATCHFLG_SENDER_SIDE
94 +           && (!for_xfer || protocol_version >= 29))
95 +               *op++ = 's';
96 +       if (match_flags & MATCHFLG_RECEIVER_SIDE
97 +           && (!for_xfer || protocol_version >= 29
98 +            || (delete_excluded && am_sender)))
99 +               *op++ = 'r';
100         if (legal_len)
101                 *op++ = ' ';
102         if (op - buf > legal_len)
103 @@ -1025,18 +1070,34 @@ char *get_rule_prefix(int match_flags, c
104  
105  static void send_rules(int f_out, struct filter_list_struct *flp)
106  {
107 -       struct filter_struct *ent;
108 +       struct filter_struct *ent, *prev = NULL;
109  
110         for (ent = flp->head; ent; ent = ent->next) {
111                 unsigned int len, plen, dlen;
112 +               int elide = 0;
113                 char *p;
114  
115 +               if (ent->match_flags & MATCHFLG_SENDER_SIDE)
116 +                       elide = am_sender ? 1 : -1;
117 +               if (ent->match_flags & MATCHFLG_RECEIVER_SIDE)
118 +                       elide = elide ? 0 : am_sender ? -1 : 1;
119 +               else if (delete_excluded)
120 +                       elide = am_sender ? 1 : -1;
121 +               if (elide < 0) {
122 +                       if (prev)
123 +                               prev->next = ent->next;
124 +                       else
125 +                               flp->head = ent->next;
126 +               } else
127 +                       prev = ent;
128 +               if (elide > 0)
129 +                       continue;
130                 if (ent->match_flags & MATCHFLG_CVS_IGNORE
131                     && !(ent->match_flags & MATCHFLG_MERGE_FILE)) {
132 -                       if (am_sender || protocol_version < 29) {
133 -                               send_rules(f_out, &cvs_filter_list);
134 +                       int f = am_sender || protocol_version < 29 ? f_out : -1;
135 +                       send_rules(f, &cvs_filter_list);
136 +                       if (f >= 0)
137                                 continue;
138 -                       }
139                 }
140                 p = get_rule_prefix(ent->match_flags, ent->pattern, 1, &plen);
141                 if (!p) {
142 @@ -1044,6 +1105,8 @@ static void send_rules(int f_out, struct
143                                 "filter rules are too modern for remote rsync.\n");
144                         exit_cleanup(RERR_SYNTAX);
145                 }
146 +               if (f_out < 0)
147 +                       continue;
148                 len = strlen(ent->pattern);
149                 dlen = ent->match_flags & MATCHFLG_DIRECTORY ? 1 : 0;
150                 if (!(plen + len + dlen))
151 @@ -1055,12 +1118,14 @@ static void send_rules(int f_out, struct
152                 if (dlen)
153                         write_byte(f_out, '/');
154         }
155 +       flp->tail = prev;
156  }
157  
158  /* This is only called by the client. */
159  void send_filter_list(int f_out)
160  {
161 -       int receiver_wants_list = delete_mode && !delete_excluded;
162 +       int receiver_wants_list = delete_mode
163 +               && (!delete_excluded || protocol_version >= 29);
164  
165         if (local_server || (am_sender && !receiver_wants_list))
166                 f_out = -1;
167 @@ -1075,10 +1140,10 @@ void send_filter_list(int f_out)
168         if (list_only == 1 && !recurse)
169                 parse_rule(&filter_list, "/*/*", MATCHFLG_NO_PREFIXES, 0);
170  
171 -       if (f_out >= 0) {
172 -               send_rules(f_out, &filter_list);
173 +       send_rules(f_out, &filter_list);
174 +
175 +       if (f_out >= 0)
176                 write_int(f_out, 0);
177 -       }
178  
179         if (cvs_exclude) {
180                 if (!am_sender || protocol_version < 29)
181 @@ -1093,8 +1158,9 @@ void recv_filter_list(int f_in)
182  {
183         char line[MAXPATHLEN+MAX_RULE_PREFIX+1]; /* +1 for trailing slash. */
184         int xflags = protocol_version >= 29 ? 0 : XFLG_OLD_PREFIXES;
185 +       int receiver_wants_list = delete_mode
186 +               && (!delete_excluded || protocol_version >= 29);
187         unsigned int len;
188 -       int receiver_wants_list = delete_mode && !delete_excluded;
189  
190         if (!local_server && (am_sender || receiver_wants_list)) {
191                 while ((len = read_int(f_in)) != 0) {
192 @@ -1111,4 +1177,7 @@ void recv_filter_list(int f_in)
193                 if (local_server || am_sender)
194                         parse_rule(&filter_list, "-C", 0, 0);
195         }
196 +
197 +       if (local_server) /* filter out any rules that aren't for us. */
198 +               send_rules(-1, &filter_list);
199  }
200 --- orig/flist.c        2005-02-12 18:39:09
201 +++ flist.c     2005-02-12 19:08:31
202 @@ -64,7 +64,6 @@ extern int copy_links;
203  extern int copy_unsafe_links;
204  extern int protocol_version;
205  extern int sanitize_paths;
206 -extern int delete_excluded;
207  extern int max_delete;
208  extern int orig_umask;
209  extern int list_only;
210 @@ -976,11 +975,7 @@ void send_file_name(int f, struct file_l
211         struct file_struct *file;
212         char fbuf[MAXPATHLEN];
213  
214 -       /* f is set to -1 when calculating deletion file list */
215 -       file = make_file(fname, flist,
216 -           f == -1 && delete_excluded? SERVER_FILTERS : ALL_FILTERS);
217 -
218 -       if (!file)
219 +       if (!(file = make_file(fname, flist, ALL_FILTERS)))
220                 return;
221  
222         maybe_emit_filelist_progress(flist);
223 --- orig/rsync.h        2005-02-07 20:41:57
224 +++ rsync.h     2005-02-12 19:08:34
225 @@ -565,9 +565,12 @@ struct map_struct {
226  #define MATCHFLG_FINISH_SETUP  (1<<13)/* per-dir merge file needs setup */
227  #define MATCHFLG_NEGATE        (1<<14)/* rule matches when pattern does not */
228  #define MATCHFLG_CVS_IGNORE    (1<<15)/* rule was -C or :C */
229 +#define MATCHFLG_SENDER_SIDE   (1<<16)/* rule applies to the sending side */
230 +#define MATCHFLG_RECEIVER_SIDE (1<<17)/* rule applies to the receiving side */
231  
232  #define MATCHFLGS_FROM_CONTAINER (MATCHFLG_ABS_PATH | MATCHFLG_INCLUDE \
233 -                               | MATCHFLG_DIRECTORY | MATCHFLG_NEGATE)
234 +                               | MATCHFLG_DIRECTORY | MATCHFLG_SENDER_SIDE \
235 +                               | MATCHFLG_NEGATE | MATCHFLG_RECEIVER_SIDE)
236  
237  struct filter_struct {
238         struct filter_struct *next;
239 --- orig/rsync.yo       2005-02-11 23:14:49
240 +++ rsync.yo    2005-02-12 19:08:37
241 @@ -679,7 +679,9 @@ send the whole directory (e.g. "dir" or 
242  for the directory's contents (e.g. "dir/*") since the wildcard is expanded
243  by the shell and rsync thus gets a request to transfer individual files, not
244  the files' parent directory.  Files that are excluded from transfer are
245 -excluded from being deleted unless you use bf(--delete-excluded).
246 +also excluded from being deleted unless you use the bf(--delete-excluded)
247 +option or mark the rules as only matching on the sending side (see the
248 +include/exclude modifiers in the FILTER RULES section).
249  
250  This option has no effect unless directory recursion is enabled.
251  
252 @@ -726,6 +728,9 @@ See bf(--delete) (which is implied) for 
253  dit(bf(--delete-excluded)) In addition to deleting the files on the
254  receiving side that are not on the sending side, this tells rsync to also
255  delete any files on the receiving side that are excluded (see bf(--exclude)).
256 +See the FILTER RULES section for a way to make individual exclusions behave
257 +this way on the receiver, and for a way to protect files from
258 +bf(--delete-excluded).
259  See bf(--delete) (which is implied) for more details on file-deletion.
260  
261  dit(bf(--ignore-errors)) Tells bf(--delete) to go ahead and delete files
262 @@ -1255,6 +1260,10 @@ bf(exclude, -) specifies an exclude patt
263  bf(include, +) specifies an include pattern. nl()
264  bf(merge, .) specifies a merge-file to read for more rules. nl()
265  bf(dir-merge, :) specifies a per-directory merge-file. nl()
266 +bf(hide, H) specifies a pattern for hiding files from the transfer. nl()
267 +bf(show, S) files that match the pattern are not hidden. nl()
268 +bf(protect, P) specifies a pattern for protecting files from deletion. nl()
269 +bf(risk, R) files that match the pattern are not protected. nl()
270  bf(clear, !) clears the current include/exclude list (takes no arg) nl()
271  )
272  
273 @@ -1279,8 +1288,8 @@ the bf(--include-from)/bf(--exclude-from
274  
275  manpagesection(INCLUDE/EXCLUDE PATTERN RULES)
276  
277 -You can include and exclude files by specifying patterns using the "+" and
278 -"-" filter rules (as introduced in the FILTER RULES section above).
279 +You can include and exclude files by specifying patterns using the "+",
280 +"-", etc. filter rules (as introduced in the FILTER RULES section above).
281  The include/exclude rules each specify a pattern that is matched against
282  the names of the files that are going to be transferred.  These patterns
283  can take several forms:
284 @@ -1419,7 +1428,9 @@ itemize(
285    it() You may also specify any of the modifiers for the "+" or "-" rules
286    (below) in order  to have the rules that are read-in from the file
287    default to having that modifier set.  For instance, "merge,-/ .excl" would
288 -  treat the contents of .excl as absolute-path excludes.
289 +  treat the contents of .excl as absolute-path excludes,
290 +  while "dir-merge,s .filt" and ":sC" would each make all their
291 +  per-directory rules apply only on the server side.
292  )
293  
294  The following modifiers are accepted after a "+" or "-":
295 @@ -1435,7 +1446,18 @@ itemize(
296    it() A bf(C) is used to indicate that all the global CVS-exclude rules
297    should be inserted as excludes in place of the "-C".  No arg should
298    follow.
299 -  )
300 +  it() An bf(s) is used to indicate that the rule applies to the sending
301 +  side.  When a rule affects the sending side, it prevents files from
302 +  being transferred.  The default is for a rule to affect both sides
303 +  unless bf(--delete-excluded) was specified, in which case default rules
304 +  become sender-side only.  See also the hide (H) and show (S) rules,
305 +  which are an alternate way to specify server-side includes/excludes.
306 +  it() An bf(r) is used to indicate that the rule applies to the receiving
307 +  side.  When a rule affects the receiving side, it prevents files from
308 +  being deleted.  See the bf(s) modifier for more info.  See also the
309 +  protect (P) and risk (R) rules, which are an alternate way to
310 +  specify receiver-side includes/excludes.
311 +)
312  
313  Per-directory rules are inherited in all subdirectories of the directory
314  where the merge-file was found unless the 'n' modifier was used.  Each