Need to make sure that the destination file doesn't exist before we
[rsync/rsync-patches.git] / filter.diff
index 033bef7..ffaf81c 100644 (file)
@@ -7,12 +7,11 @@ This patch adds the ability to merge rules into your excludes/includes
 using a ". FILE" idiom.  If you specify a name with a preceding -p
 option, that filename will be looked for in every subdirectory that
 rsync visits, and the rules found in that subdirectory's file will
-affect either just that dir or, if -i is specified, that dir and its
-subdirs.
+affect that dir and its subdirs.
 
 For example:
 
-  rsync -av --exclude='. -pi .excl' from/ to
+  rsync -av --exclude='. -p .excl' from/ to
 
 The above will look for a file named ".excl" in every directory of the
 hierarchy that rsync visits, and it will exclude (by default) names
@@ -21,53 +20,114 @@ this:
 
   + *.c
   . -p .excl2
-  . -i .excl3
+  . .excl3
   *.o
+  /foobar
 
 Then the file ".excl2" will also be read in from the current dir and all
-its subdirs (due to the -p option) with each file's rules only affecting
-the file's current directory.  The file ".excl3" would just be read in
-for the current dir (because it was not specified with the -p option),
-but its rules would be inherited by all its subdirectories (because of
-the -i option).
+its subdirs (due to the -p option).  The file ".excl3" would just be
+read in from the current dir.   The exclusion of "foobar" will only
+happen in that .excl file's directory because the rule is anchored (so
+that's how you can make rules local instead of inherited).
 
 ..wayne..
 
---- orig/exclude.c     2004-08-05 23:16:37
-+++ exclude.c  2004-08-06 23:22:00
-@@ -27,17 +27,66 @@
- #include "rsync.h"
+--- orig/clientserver.c        2004-08-02 02:29:16
++++ clientserver.c     2004-08-10 15:44:15
+@@ -48,12 +48,14 @@ extern int no_detach;
+ extern int default_af_hint;
+ extern char *bind_address;
+ extern struct exclude_list_struct server_exclude_list;
+-extern char *exclude_path_prefix;
+ extern char *config_file;
+ extern char *files_from;
+ char *auth_user;
++/* Length of lp_path() string when in daemon mode & not chrooted, else 0. */
++unsigned int module_dirlen = 0;
++
+ /**
+  * Run a client connected to an rsyncd.  The alternative to this
+  * function for remote-shell connections is do_cmd().
+@@ -300,26 +302,28 @@ static int rsync_module(int f_in, int f_
+       /* TODO: Perhaps take a list of gids, and make them into the
+        * supplementary groups. */
+-      exclude_path_prefix = use_chroot? "" : lp_path(i);
+-      if (*exclude_path_prefix == '/' && !exclude_path_prefix[1])
+-              exclude_path_prefix = "";
++      if (use_chroot) {
++              module_dirlen = 0;
++              set_excludes_dir("/", 1);
++      } else {
++              module_dirlen = strlen(lp_path(i));
++              set_excludes_dir(lp_path(i), module_dirlen);
++      }
+       p = lp_include_from(i);
+       add_exclude_file(&server_exclude_list, p,
+-                       XFLG_FATAL_ERRORS | XFLG_DEF_INCLUDE);
++                       XFLG_FATAL_ERRORS | XFLG_DEF_INCLUDE | XFLG_ABS_PATH);
  
- extern int verbose;
-+extern int am_sender;
+       p = lp_include(i);
+       add_exclude(&server_exclude_list, p,
+-                  XFLG_WORD_SPLIT | XFLG_DEF_INCLUDE);
++                  XFLG_WORD_SPLIT | XFLG_DEF_INCLUDE | XFLG_ABS_PATH);
+       p = lp_exclude_from(i);
+       add_exclude_file(&server_exclude_list, p,
+-                       XFLG_FATAL_ERRORS);
++                       XFLG_FATAL_ERRORS | XFLG_ABS_PATH);
+       p = lp_exclude(i);
+-      add_exclude(&server_exclude_list, p, XFLG_WORD_SPLIT);
+-
+-      exclude_path_prefix = NULL;
++      add_exclude(&server_exclude_list, p, XFLG_WORD_SPLIT | XFLG_ABS_PATH);
+       log_init();
+--- orig/exclude.c     2004-08-10 18:17:01
++++ exclude.c  2004-08-13 07:40:08
+@@ -30,13 +30,69 @@ extern int verbose;
  extern int eol_nulls;
  extern int list_only;
  extern int recurse;
 +extern int io_error;
-+extern int module_id;
-+extern int delete_mode;
-+extern int delete_excluded;
 +extern int sanitize_paths;
  
  extern char curr_dir[];
 +extern unsigned int curr_dir_len;
++extern unsigned int module_dirlen;
  
  struct exclude_list_struct exclude_list = { 0, 0, "" };
 -struct exclude_list_struct local_exclude_list = { 0, 0, "per-dir .cvsignore " };
  struct exclude_list_struct server_exclude_list = { 0, 0, "server " };
- char *exclude_path_prefix = NULL;
+-char *exclude_path_prefix = NULL;
++
 +struct mergelist_save_struct {
 +    struct exclude_list_struct *array;
 +    int count;
 +};
 +
-+static char dirbuf[MAXPATHLEN];
-+static unsigned int dirbuf_offset = 0;
++/* The dirbuf is set by push_local_excludes() to the current subdirectory
++ * relative to curr_dir that is being processed.  The path always has a
++ * trailing slash appended, and the variable dirbuf_len contains the length
++ * of this path prefix.  The path is always absolute. */
++static char dirbuf[MAXPATHLEN+1];
++static unsigned int dirbuf_len = 0;
++static int dirbuf_depth;
++
++/* This is True when we're scanning parent dirs for per-dir merge-files. */
++static BOOL parent_dirscan = False;
++
++/* This array contains a list of all the currently active per-dir merge
++ * files.  This makes it easier to save the appropriate values when we
++ * "push" down into each subdirectory. */
 +static struct exclude_struct **mergelist_parents;
 +static int mergelist_cnt = 0;
 +static int mergelist_size = 0;
-+static int initialized_mergelist_cnt = -1;
 +
 +/* Each exclude_list_struct describes a singly-linked list by keeping track
 + * of both the head and tail pointers.  The list is slightly unusual in that
@@ -99,11 +159,10 @@ the -i option).
 + * the list for a new local dir, we just save off the exclude_list_struct
 + * values (so we can pop back to them later) and set the tail to NULL.
 + */
-+
  /** Build an exclude structure given an exclude pattern. */
  static void make_exclude(struct exclude_list_struct *listp, const char *pat,
-                        unsigned int pat_len, unsigned int mflags)
-@@ -46,6 +95,31 @@ static void make_exclude(struct exclude_
+@@ -46,23 +102,50 @@ static void make_exclude(struct exclude_
        const char *cp;
        unsigned int ex_len;
  
@@ -127,7 +186,7 @@ the -i option).
 +              if ((pat_len == 10 || (pat_len > 10 && pat[pat_len-11] == '/'))
 +                  && strncmp(pat+pat_len-10, ".cvsignore", 10) == 0) {
 +                      mflags |= MATCHFLG_CVSIGNORE;
-+                      mflags &= ~(MATCHFLG_INCLUDE | MATCHFLG_INHERIT);
++                      mflags &= ~MATCHFLG_INCLUDE;
 +              } else
 +                      mflags &= ~MATCHFLG_CVSIGNORE;
 +      }
@@ -135,7 +194,32 @@ the -i option).
        ret = new(struct exclude_struct);
        if (!ret)
                out_of_memory("make_exclude");
-@@ -81,14 +155,40 @@ static void make_exclude(struct exclude_
+       memset(ret, 0, sizeof ret[0]);
+-      if (exclude_path_prefix)
+-              mflags |= MATCHFLG_ABS_PATH;
+-      if (exclude_path_prefix && *pat == '/')
+-              ex_len = strlen(exclude_path_prefix);
+-      else
++      if (mflags & MATCHFLG_ABS_PATH) {
++              if (*pat != '/') {
++                      mflags &= ~MATCHFLG_ABS_PATH;
++                      ex_len = 0;
++              } else
++                      ex_len = dirbuf_len - module_dirlen - 1;
++      } else
+               ex_len = 0;
+       ret->pattern = new_array(char, ex_len + pat_len + 1);
+       if (!ret->pattern)
+               out_of_memory("make_exclude");
+       if (ex_len)
+-              memcpy(ret->pattern, exclude_path_prefix, ex_len);
++              memcpy(ret->pattern, dirbuf + module_dirlen, ex_len);
+       strlcpy(ret->pattern + ex_len, pat, pat_len + 1);
+       pat_len += ex_len;
+@@ -81,14 +164,40 @@ static void make_exclude(struct exclude_
                mflags |= MATCHFLG_DIRECTORY;
        }
  
@@ -180,7 +264,7 @@ the -i option).
                listp->tail->next = ret;
                listp->tail = ret;
        }
-@@ -96,22 +196,247 @@ static void make_exclude(struct exclude_
+@@ -96,22 +205,267 @@ static void make_exclude(struct exclude_
  
  static void free_exclude(struct exclude_struct *ex)
  {
@@ -214,140 +298,159 @@ the -i option).
        listp->head = listp->tail = NULL;
  }
  
-+/* This returns a fully expanded filename for the merge-file name if the
-+ * name has any slashes in it, otherwise it returns the original name. */
-+static char *parse_merge_name(const char *merge_file, unsigned int *len_ptr)
++/* This returns an expanded (absolute) filename for the merge-file name if
++ * the name has any slashes in it OR if the parent_dirscan var is True;
++ * otherwise it returns the original merge_file name.  If the len_ptr value
++ * is non-NULL the merge_file name is limited by the referenced length
++ * value and will be updated with the length of the resulting name.  We
++ * always return a name that is null terminated, even if the merge_file
++ * name was not. */
++static char *parse_merge_name(const char *merge_file, unsigned int *len_ptr,
++                            unsigned int prefix_skip)
 +{
 +      static char buf[MAXPATHLEN];
 +      char *fn, tmpbuf[MAXPATHLEN];
-+      unsigned int fn_len, cd_len;
++      unsigned int fn_len;
 +
-+      cd_len = dirbuf_offset && *dirbuf == '/' ? 0 : curr_dir_len + 1;
-+      if (cd_len && *merge_file != '/') {
-+              const char *p;
++      if (!parent_dirscan && *merge_file != '/') {
++              /* Return the name unchanged it doesn't have any slashes. */
 +              if (len_ptr) {
-+                      for (p = merge_file + *len_ptr;
-+                           --p > merge_file && *p != '/'; ) {}
-+              } else
-+                      p = strchr(merge_file, '/');
-+              if (!p || p == merge_file)
++                      const char *p = merge_file + *len_ptr;
++                      while (--p > merge_file && *p != '/') {}
++                      if (p == merge_file) {
++                              strlcpy(buf, merge_file, *len_ptr + 1);
++                              return buf;
++                      }
++              } else if (strchr(merge_file, '/') == NULL)
 +                      return (char *)merge_file;
 +      }
 +
 +      fn = *merge_file == '/' ? buf : tmpbuf;
 +      if (sanitize_paths) {
++              const char *r = prefix_skip ? "/" : NULL;
 +              /* null-terminate the name if it isn't already */
 +              if (len_ptr && merge_file[*len_ptr]) {
 +                      char *to = fn == buf ? tmpbuf : buf;
 +                      strlcpy(to, merge_file, *len_ptr + 1);
 +                      merge_file = to;
 +              }
-+              dirbuf[dirbuf_offset] = '\0';
-+              if (!sanitize_path(fn, merge_file, dirbuf)) {
-+                      rprintf(FERROR, "merge filename overflows: %s\n",
++              if (!sanitize_path(fn, merge_file, r, dirbuf_depth)) {
++                      rprintf(FERROR, "merge-file name overflows: %s\n",
 +                              merge_file);
 +                      return NULL;
 +              }
 +      } else {
 +              strlcpy(fn, merge_file, len_ptr ? *len_ptr + 1 : MAXPATHLEN);
-+              clean_fname(fn);
++              clean_fname(fn, 1);
 +      }
 +      
-+      if (*fn == '/')
-+              return fn;
-+
 +      fn_len = strlen(fn);
-+      if (cd_len + dirbuf_offset + fn_len >= MAXPATHLEN) {
-+              rprintf(FERROR, "merge filename overflows: %s\n", fn);
++      if (fn == buf)
++              goto done;
++
++      if (dirbuf_len + fn_len >= MAXPATHLEN) {
++              rprintf(FERROR, "merge-file name overflows: %s\n", fn);
 +              return NULL;
 +      }
-+      if (cd_len) {
-+              memcpy(buf, curr_dir, curr_dir_len);
-+              buf[curr_dir_len] = '/';
-+      }
-+      if (dirbuf_offset)
-+              memcpy(buf + cd_len, dirbuf, dirbuf_offset);
-+      memcpy(buf + cd_len + dirbuf_offset, fn, fn_len + 1);
-+      fn_len = clean_fname(buf);
++      memcpy(buf, dirbuf + prefix_skip, dirbuf_len - prefix_skip);
++      memcpy(buf + dirbuf_len - prefix_skip, fn, fn_len + 1);
++      fn_len = clean_fname(buf, 1);
++
++    done:
 +      if (len_ptr)
 +              *len_ptr = fn_len;
-+
 +      return buf;
 +}
 +
-+static void prep_merge_file(struct exclude_struct *ex,
-+                          struct exclude_list_struct *lp, int flags,
-+                          char *dir, unsigned int dirlen)
++/* Sets the dirbuf and dirbuf_len values. */
++void set_excludes_dir(const char *dir, unsigned int dirlen)
++{
++      unsigned int len;
++      if (*dir != '/') {
++              memcpy(dirbuf, curr_dir, curr_dir_len);
++              dirbuf[curr_dir_len] = '/';
++              len = curr_dir_len + 1;
++              if (len + dirlen >= MAXPATHLEN)
++                      dirlen = 0;
++      } else
++              len = 0;
++      memcpy(dirbuf + len, dir, dirlen);
++      dirbuf[dirlen + len] = '\0';
++      dirbuf_len = clean_fname(dirbuf, 1);
++      if (dirbuf_len > 1 && dirbuf[dirbuf_len-1] == '.'
++          && dirbuf[dirbuf_len-2] == '/')
++              dirbuf_len -= 2;
++      dirbuf[dirbuf_len++] = '/';
++      dirbuf[dirbuf_len] = '\0';
++      if (sanitize_paths)
++              dirbuf_depth = count_dir_elements(dirbuf + module_dirlen);
++}
++
++/* This routine takes a per-dir merge-file entry and finishes its setup.
++ * If the name has a path portion then we check to see if it refers to a
++ * parent directory of the first transfer dir.  If it does, we scan all the
++ * dirs from that point through the parent dir of the transfer dir looking
++ * for the per-dir merge-file in each one. */
++static BOOL setup_merge_file(struct exclude_struct *ex,
++                           struct exclude_list_struct *lp, int flags)
 +{
 +      char buf[MAXPATHLEN];
-+      char *x, *y, *fn = ex->pattern;
-+
-+      if (!(fn = parse_merge_name(fn, NULL)) || *fn != '/')
-+              return;
-+
-+      x = strrchr(fn, '/');
-+      *x = '\0';
-+      if (dir[dirlen]) /* avoid writing to a read-only string */
-+              dir[dirlen] = '\0';
-+      pathjoin(dirbuf, MAXPATHLEN, sanitize_paths ? "" : curr_dir, dir);
-+      if (dirlen == 2 && *dir == '.') {
-+              int len = strlen(dirbuf);
-+              dirbuf[len-2] = '\0';
-+      }
-+      if (!*fn)
-+              fn = "/";
-+      if (*fn == '/')
-+              strlcpy(buf, fn, MAXPATHLEN);
++      char *x, *y, *pat = ex->pattern;
++      unsigned int len;
++
++      if (!(x = parse_merge_name(pat, NULL, 0)) || *x != '/')
++              return 0;
++
++      y = strrchr(x, '/');
++      *y = '\0';
++      ex->pattern = strdup(y+1);
++      if (!*x)
++              x = "/";
++      if (*x == '/')
++              strlcpy(buf, x, MAXPATHLEN);
 +      else
-+              pathjoin(buf, MAXPATHLEN, dirbuf, fn);
-+      fn = x + 1;
++              pathjoin(buf, MAXPATHLEN, dirbuf, x);
 +
-+      clean_fname(buf);
-+      if (!*buf || buf[1])
-+              strlcat(buf, "/", MAXPATHLEN);
-+      x = buf;
-+      if (sanitize_paths)
-+              x += strlen(lp_path(module_id));
-+      for (y = dirbuf; *x && *x == *y; x++, y++) {}
++      len = clean_fname(buf, 1);
++      if (len != 1 && len < MAXPATHLEN-1) {
++              buf[len++] = '/';
++              buf[len] = '\0';
++      }
++      /* This ensures that the specified dir is a parent of the transfer. */
++      for (x = buf, y = dirbuf; *x && *x == *y; x++, y++) {}
 +      if (*x)
-+              y += strlen(y);
++              y += strlen(y); /* nope -- skip the scan */
 +
++      parent_dirscan = True;
 +      while (*y) {
 +              char save[MAXPATHLEN];
 +              strlcpy(save, y, MAXPATHLEN);
-+              dirbuf_offset = y - dirbuf;
-+              strlcpy(x, fn, MAXPATHLEN - (x - buf));
++              *y = '\0';
++              dirbuf_len = y - dirbuf;
++              strlcpy(x, ex->pattern, MAXPATHLEN - (x - buf));
++              add_exclude_file(lp, buf, flags | XFLG_ABS_PATH);
++              if (ex->match_flags & MATCHFLG_CVSIGNORE)
++                      lp->head = NULL; /* CVS doesn't inherit rules. */
 +              lp->tail = NULL;
-+              add_exclude_file(lp, buf, flags);
 +              strlcpy(y, save, MAXPATHLEN);
 +              while ((*x++ = *y++) != '/') {}
 +      }
-+
-+      if (dirlen == 2 && *dir == '.') /* dir[dirlen-1] is always '/' */
-+              dirbuf_offset = 0;
-+      else
-+              dirbuf_offset = dirlen;
-+      memcpy(dirbuf, dir, dirbuf_offset);
-+
-+      x = ex->pattern;
-+      ex->pattern = strdup(fn);
-+      free(x);
++      parent_dirscan = False;
++      free(pat);
++      return 1;
 +}
 +
-+void *push_local_excludes(char *dir, unsigned int dirlen)
++/* Each time rsync changes to a new directory it call this function to
++ * handle all the per-dir merge-files.  The "dir" value is the current path
++ * relative to curr_dir (which might not be null-terminated).  We copy it
++ * into dirbuf so that we can easily append a file name on the end. */
++void *push_local_excludes(const char *dir, unsigned int dirlen)
 +{
 +      struct mergelist_save_struct *push;
 +      struct exclude_list_struct *ap;
 +      int i;
 +
-+      /* Make it easy to construct the full path for a merge-file that was
-+       * specified with a relative path by saving off the current dir. */
-+      if (dirlen == 2 && *dir == '.') /* dir[dirlen-1] is always '/' */
-+              dirbuf_offset = 0;
-+      else
-+              dirbuf_offset = dirlen;
-+      memcpy(dirbuf, dir, dirbuf_offset);
-+
++      set_excludes_dir(dir, dirlen);
 +
 +      if (!(push = new_array(struct mergelist_save_struct, 1)))
 +              out_of_memory("push_local_excludes");
@@ -374,30 +477,31 @@ the -i option).
 +                              who_am_i(), lp->debug_type);
 +              }
 +
-+              if (ex->match_flags & MATCHFLG_CVSIGNORE)
++              if (ex->match_flags & MATCHFLG_CVSIGNORE) {
++                      lp->head = NULL; /* CVS doesn't inherit rules. */
 +                      flags = XFLG_WORD_SPLIT | XFLG_WORDS_ONLY;
-+              else {
++              else {
 +                      flags = ex->match_flags & MATCHFLG_INCLUDE
 +                          ? XFLG_DEF_INCLUDE : 0;
 +              }
++              lp->tail = NULL; /* Switch any local rules to inherited. */
 +
-+              if (initialized_mergelist_cnt < i) {
-+                      prep_merge_file(ex, lp, flags, dir, dirlen);
-+                      initialized_mergelist_cnt = i;
++              if (ex->match_flags & MATCHFLG_FINISH_SETUP) {
++                      ex->match_flags &= ~MATCHFLG_FINISH_SETUP;
++                      if (setup_merge_file(ex, lp, flags))
++                              set_excludes_dir(dir, dirlen);
 +              }
 +
-+              if (!(ex->match_flags & MATCHFLG_INHERIT))
-+                      lp->head = NULL;
-+              lp->tail = NULL;
-+              if (strlcpy(dirbuf +  dirbuf_offset, ex->pattern,
-+                  MAXPATHLEN - dirbuf_offset) < MAXPATHLEN - dirbuf_offset)
-+                      add_exclude_file(lp, dirbuf, flags);
++              if (strlcpy(dirbuf + dirbuf_len, ex->pattern,
++                  MAXPATHLEN - dirbuf_len) < MAXPATHLEN - dirbuf_len)
++                      add_exclude_file(lp, dirbuf, flags | XFLG_ABS_PATH);
 +              else {
 +                      io_error |= IOERR_GENERAL;
 +                      rprintf(FINFO,
 +                          "cannot add local excludes in long-named directory %s\n",
 +                          full_fname(dirbuf));
 +              }
++              dirbuf[dirbuf_len] = '\0';
 +      }
 +
 +      return (void*)push;
@@ -421,7 +525,7 @@ the -i option).
 +              clear_exclude_list(lp);
 +      }
 +
-+      mergelist_cnt = initialized_mergelist_cnt = pop->count;
++      mergelist_cnt = pop->count;
 +      for (i = 0, ap = pop->array; i < mergelist_cnt; i++) {
 +              memcpy(mergelist_parents[i]->u.mergelist, ap++,
 +                     sizeof (struct exclude_list_struct));
@@ -434,7 +538,7 @@ the -i option).
  static int check_one_exclude(char *name, struct exclude_struct *ex,
                               int name_is_dir)
  {
-@@ -122,7 +447,7 @@ static int check_one_exclude(char *name,
+@@ -125,13 +479,14 @@ static int check_one_exclude(char *name,
        /* If the pattern does not have any slashes AND it does not have
         * a "**" (which could match a slash), then we just match the
         * name portion of the path. */
@@ -443,17 +547,16 @@ the -i option).
                if ((p = strrchr(name,'/')) != NULL)
                        name = p+1;
        }
-@@ -133,7 +458,8 @@ static int check_one_exclude(char *name,
+       else if (ex->match_flags & MATCHFLG_ABS_PATH && *name != '/'
+-          && curr_dir[1]) {
+-              pathjoin(full_name, sizeof full_name, curr_dir + 1, name);
++          && curr_dir_len > module_dirlen + 1) {
++              pathjoin(full_name, sizeof full_name,
++                       curr_dir + module_dirlen + 1, name);
                name = full_name;
        }
  
--      if (!name[0]) return 0;
-+      if (!name[0])
-+              return 0;
-       if (ex->match_flags & MATCHFLG_DIRECTORY && !name_is_dir)
-               return 0;
-@@ -148,9 +474,9 @@ static int check_one_exclude(char *name,
+@@ -148,9 +503,9 @@ static int check_one_exclude(char *name,
        if (ex->match_flags & MATCHFLG_WILD) {
                /* A non-anchored match with an infix slash and no "**"
                 * needs to match the last slash_cnt+1 name elements. */
@@ -465,7 +568,7 @@ the -i option).
                        for (p = name + strlen(name) - 1; p >= name; p--) {
                                if (*p == '/' && !--cnt)
                                        break;
-@@ -221,6 +547,13 @@ int check_exclude(struct exclude_list_st
+@@ -221,6 +576,13 @@ int check_exclude(struct exclude_list_st
        struct exclude_struct *ent;
  
        for (ent = listp->head; ent; ent = ent->next) {
@@ -479,7 +582,7 @@ the -i option).
                if (check_one_exclude(name, ent, name_is_dir)) {
                        report_exclude_result(name, ent, name_is_dir,
                                              listp->debug_type);
-@@ -253,12 +586,41 @@ static const char *get_exclude_tok(const
+@@ -253,11 +615,36 @@ static const char *get_exclude_tok(const
                p = (const char *)s;
        }
  
@@ -498,11 +601,9 @@ the -i option).
 +                              s += 2;
 +                              do {
 +                                      switch (*++s) {
-+                                      case 'i':
-+                                              mflags |= MATCHFLG_INHERIT;
-+                                              break;
 +                                      case 'p':
-+                                              mflags |= MATCHFLG_PERDIR_MERGE;
++                                              mflags |= MATCHFLG_PERDIR_MERGE
++                                                      | MATCHFLG_FINISH_SETUP;
 +                                              break;
 +                                      case '-':
 +                                              if (s[1] == ' ')
@@ -518,12 +619,27 @@ the -i option).
 +              }
 +          done:
                s += 2;
-+              if (mflags & MATCHFLG_MERGE_FILE && *s == '.' && s[1] == '/')
-+                      s += 2;
        } else if (xflags & XFLG_DEF_INCLUDE)
                mflags |= MATCHFLG_INCLUDE;
+@@ -273,6 +660,8 @@ static const char *get_exclude_tok(const
+       if (*p == '!' && len == 1 && !(xflags & XFLG_WORDS_ONLY))
+               mflags |= MATCHFLG_CLEAR_LIST;
++      if (xflags & XFLG_ABS_PATH)
++              mflags |= MATCHFLG_ABS_PATH;
+       *len_ptr = len;
+       *flag_ptr = mflags;
+@@ -284,7 +673,7 @@ void add_exclude(struct exclude_list_str
+                int xflags)
+ {
+       unsigned int pat_len, mflags;
+-      const char *cp;
++      const char *cp, *p;
  
-@@ -292,9 +654,15 @@ void add_exclude(struct exclude_list_str
+       if (!pattern)
+               return;
+@@ -292,9 +681,15 @@ void add_exclude(struct exclude_list_str
        cp = pattern;
        pat_len = 0;
        while (1) {
@@ -539,7 +655,7 @@ the -i option).
  
                if (mflags & MATCHFLG_CLEAR_LIST) {
                        if (verbose > 2) {
-@@ -306,13 +674,23 @@ void add_exclude(struct exclude_list_str
+@@ -306,13 +701,24 @@ void add_exclude(struct exclude_list_str
                        continue;
                }
  
@@ -550,17 +666,18 @@ the -i option).
 -                              who_am_i(), (int)pat_len, cp, listp->debug_type,
 -                              mflags & MATCHFLG_INCLUDE ? "in" : "ex");
 +              if (mflags & MATCHFLG_MERGE_FILE) {
++                      unsigned int len = pat_len;
 +                      if (mflags & MATCHFLG_PERDIR_MERGE) {
-+                              if (dirbuf_offset && *dirbuf == '/') {
-+                                      if (!(cp = parse_merge_name(cp, &pat_len)))
++                              if (parent_dirscan) {
++                                      if (!(p = parse_merge_name(cp, &len, module_dirlen)))
 +                                              continue;
-+                                      make_exclude(listp, cp, pat_len, mflags);
++                                      make_exclude(listp, p, len, mflags);
 +                                      continue;
 +                              }
 +                      } else {
-+                              if (!(cp = parse_merge_name(cp, &pat_len)))
++                              if (!(p = parse_merge_name(cp, &len, 0)))
 +                                      continue;
-+                              add_exclude_file(listp, cp, xflags | XFLG_FATAL_ERRORS);
++                              add_exclude_file(listp, p, xflags | XFLG_FATAL_ERRORS);
 +                              continue;
 +                      }
                }
@@ -569,7 +686,7 @@ the -i option).
        }
  }
  
-@@ -321,7 +699,7 @@ void add_exclude_file(struct exclude_lis
+@@ -321,7 +727,7 @@ void add_exclude_file(struct exclude_lis
                      int xflags)
  {
        FILE *fp;
@@ -578,19 +695,20 @@ the -i option).
        char *eob = line + sizeof line - 1;
        int word_split = xflags & XFLG_WORD_SPLIT;
  
-@@ -343,6 +721,11 @@ void add_exclude_file(struct exclude_lis
+@@ -342,6 +748,12 @@ void add_exclude_file(struct exclude_lis
+               }
                return;
        }
++      dirbuf[dirbuf_len] = '\0';
++
 +      if (verbose > 2) {
 +              rprintf(FINFO, "[%s] add_exclude_file(%s,%d)\n",
 +                      who_am_i(), fname, xflags);
 +      }
-+
        while (1) {
                char *s = line;
-               int ch, overflow = 0;
-@@ -402,7 +785,21 @@ void send_exclude_list(int f)
+@@ -402,7 +814,21 @@ void send_exclude_list(int f)
                if (ent->match_flags & MATCHFLG_INCLUDE) {
                        write_int(f, l + 2);
                        write_buf(f, "+ ", 2);
@@ -602,8 +720,8 @@ the -i option).
 +                      if (ent->match_flags & MATCHFLG_PERDIR_MERGE) {
 +                              *op++ = '-';
 +                              *op++ = 'p';
-+                              if (ent->match_flags & MATCHFLG_INHERIT)
-+                                      *op++ = 'i';
++                              if (*p == '-')
++                                      *op++ = '-';
 +                              *op++ = ' ';
 +                      }
 +                      write_int(f, l + (op - buf));
@@ -613,7 +731,7 @@ the -i option).
                        write_int(f, l + 2);
                        write_buf(f, "- ", 2);
                } else
-@@ -443,6 +840,7 @@ void add_cvs_excludes(void)
+@@ -443,6 +869,7 @@ void add_cvs_excludes(void)
        char fname[MAXPATHLEN];
        char *p;
  
@@ -621,9 +739,9 @@ the -i option).
        add_exclude(&exclude_list, default_cvsignore,
                    XFLG_WORD_SPLIT | XFLG_WORDS_ONLY);
  
---- orig/flist.c       2004-08-05 21:57:29
-+++ flist.c    2004-08-06 20:52:26
-@@ -39,10 +39,9 @@ extern int module_id;
+--- orig/flist.c       2004-09-21 09:40:27
++++ flist.c    2004-08-12 18:59:28
+@@ -40,10 +40,9 @@ extern int module_id;
  extern int ignore_errors;
  extern int numeric_ids;
  
@@ -635,7 +753,7 @@ the -i option).
  extern char *files_from;
  extern int filesfrom_fd;
  
-@@ -66,7 +65,6 @@ extern int list_only;
+@@ -67,7 +66,6 @@ extern int list_only;
  
  extern struct exclude_list_struct exclude_list;
  extern struct exclude_list_struct server_exclude_list;
@@ -643,7 +761,7 @@ the -i option).
  
  int io_error;
  
-@@ -221,8 +219,6 @@ int link_stat(const char *path, STRUCT_S
+@@ -223,8 +221,6 @@ int link_stat(const char *path, STRUCT_S
   */
  static int check_exclude_file(char *fname, int is_dir, int exclude_level)
  {
@@ -652,7 +770,7 @@ the -i option).
  #if 0 /* This currently never happens, so avoid a useless compare. */
        if (exclude_level == NO_EXCLUDES)
                return 0;
-@@ -244,10 +240,7 @@ static int check_exclude_file(char *fnam
+@@ -246,10 +242,7 @@ static int check_exclude_file(char *fnam
        if (exclude_level != ALL_EXCLUDES)
                return 0;
        if (exclude_list.head
@@ -664,7 +782,7 @@ the -i option).
                return 1;
        return 0;
  }
-@@ -954,15 +947,7 @@ void send_file_name(int f, struct file_l
+@@ -978,15 +971,7 @@ void send_file_name(int f, struct file_l
  
        if (recursive && S_ISDIR(file->mode)
            && !(file->flags & FLAG_MOUNT_POINT)) {
@@ -680,7 +798,7 @@ the -i option).
        }
  }
  
-@@ -973,6 +958,7 @@ static void send_directory(int f, struct
+@@ -997,6 +982,7 @@ static void send_directory(int f, struct
        struct dirent *di;
        char fname[MAXPATHLEN];
        unsigned int offset;
@@ -688,7 +806,7 @@ the -i option).
        char *p;
  
        d = opendir(dir);
-@@ -996,18 +982,7 @@ static void send_directory(int f, struct
+@@ -1020,18 +1006,7 @@ static void send_directory(int f, struct
                offset++;
        }
  
@@ -708,7 +826,7 @@ the -i option).
  
        for (errno = 0, di = readdir(d); di; errno = 0, di = readdir(d)) {
                char *dname = d_name(di);
-@@ -1028,6 +1003,8 @@ static void send_directory(int f, struct
+@@ -1052,6 +1027,8 @@ static void send_directory(int f, struct
                rsyserr(FERROR, errno, "readdir(%s)", dir);
        }
  
@@ -717,55 +835,52 @@ the -i option).
        closedir(d);
  }
  
-@@ -1047,6 +1024,7 @@ struct file_list *send_file_list(int f, 
+@@ -1071,6 +1048,7 @@ struct file_list *send_file_list(int f, 
        char *p, *dir, olddir[sizeof curr_dir];
        char lastpath[MAXPATHLEN] = "";
        struct file_list *flist;
-+      void *first_push = NULL;
++      BOOL need_first_push = True;
        int64 start_write;
        int use_ff_fd = 0;
  
-@@ -1067,6 +1045,14 @@ struct file_list *send_file_list(int f, 
+@@ -1091,6 +1069,10 @@ struct file_list *send_file_list(int f, 
                                exit_cleanup(RERR_FILESELECT);
                        }
                        use_ff_fd = 1;
 +                      if (curr_dir_len < MAXPATHLEN - 1) {
-+                              char buf[MAXPATHLEN];
-+                              strcpy(buf, curr_dir);
-+                              buf[curr_dir_len] = '/';
-+                              buf[curr_dir_len+1] = '\0';
-+                              first_push = push_local_excludes(buf,
-+                                  curr_dir_len+1);
++                              push_local_excludes(curr_dir, curr_dir_len);
++                              need_first_push = False;
 +                      }
                }
        }
  
-@@ -1097,6 +1083,22 @@ struct file_list *send_file_list(int f, 
+@@ -1121,6 +1103,15 @@ struct file_list *send_file_list(int f, 
                        }
                }
  
-+              if (!first_push) {
-+                      char ch;
++              if (need_first_push) {
 +                      if ((p = strrchr(fname, '/')) != NULL) {
-+                              dir = fname;
-+                              p++;
-+                      } else {
-+                              dir = "./";
-+                              p = dir + 2;
-+                      }
-+                      if ((ch = *p) != '\0')
-+                              *p = '\0';
-+                      first_push = push_local_excludes(dir, p - dir);
-+                      if (ch)
-+                              *p = ch;
++                              if (*++p && strcmp(p, ".") != 0)
++                                      push_local_excludes(fname, p - fname);
++                      } else if (strcmp(fname, ".") != 0)
++                              push_local_excludes(fname, 0);
++                      need_first_push = False;
 +              }
 +
                if (link_stat(fname, &st, keep_dirlinks) != 0) {
                        if (f != -1) {
                                io_error |= IOERR_GENERAL;
---- orig/options.c     2004-08-05 21:57:29
-+++ options.c  2004-08-05 21:57:19
-@@ -389,6 +389,7 @@ static struct poptOption long_options[] 
+--- orig/options.c     2004-09-20 05:10:48
++++ options.c  2004-08-12 18:59:28
+@@ -287,6 +287,7 @@ void usage(enum logcode F)
+   rprintf(F,"     --include=PATTERN       don't exclude files matching PATTERN\n");
+   rprintf(F,"     --include-from=FILE     don't exclude patterns listed in FILE\n");
+   rprintf(F,"     --files-from=FILE       read FILE for list of source-file names\n");
++  rprintf(F," -E                          same as --exclude='. -p /.rsync-excludes'\n");
+   rprintf(F," -0, --from0                 all *-from file lists are delimited by nulls\n");
+   rprintf(F,"     --version               print version number\n");
+   rprintf(F,"     --daemon                run as an rsync daemon\n");
+@@ -389,6 +390,7 @@ static struct poptOption long_options[] 
    {"ignore-errors",    0,  POPT_ARG_NONE,   &ignore_errors, 0, 0, 0 },
    {"blocking-io",      0,  POPT_ARG_VAL,    &blocking_io, 1, 0, 0 },
    {"no-blocking-io",   0,  POPT_ARG_VAL,    &blocking_io, 0, 0, 0 },
@@ -773,28 +888,36 @@ the -i option).
    {0,                 'P', POPT_ARG_NONE,   0,              'P', 0, 0 },
    {"config",           0,  POPT_ARG_STRING, &config_file, 0, 0, 0 },
    {"port",             0,  POPT_ARG_INT,    &rsync_port, 0, 0, 0 },
-@@ -589,6 +590,11 @@ int parse_arguments(int *argc, const cha
+@@ -585,6 +587,11 @@ int parse_arguments(int *argc, const cha
                        am_sender = 1;
                        break;
  
 +              case 'E':
 +                      add_exclude(&exclude_list,
-+                                  ". -ip /.rsync-excludes", 0);
++                                  ". -p /.rsync-excludes", 0);
 +                      break;
 +
                case 'P':
                        do_progress = 1;
                        keep_partial = 1;
 --- orig/rsync.h       2004-08-03 15:41:32
-+++ rsync.h    2004-08-06 06:23:10
-@@ -499,11 +499,18 @@ struct map_struct {
++++ rsync.h    2004-08-08 06:07:01
+@@ -108,6 +108,7 @@
+ #define XFLG_DEF_INCLUDE      (1<<1)
+ #define XFLG_WORDS_ONLY       (1<<2)
+ #define XFLG_WORD_SPLIT       (1<<3)
++#define XFLG_ABS_PATH         (1<<4)
+ #define PERMS_REPORT          (1<<0)
+ #define PERMS_SKIP_MTIME      (1<<1)
+@@ -499,11 +500,18 @@ struct map_struct {
  #define MATCHFLG_INCLUDE      (1<<4) /* this is an include, not an exclude */
  #define MATCHFLG_DIRECTORY    (1<<5) /* this matches only directories */
  #define MATCHFLG_CLEAR_LIST   (1<<6) /* this item is the "!" token */
 +#define MATCHFLG_MERGE_FILE   (1<<7) /* specifies a file to merge */
 +#define MATCHFLG_CVSIGNORE    (1<<8) /* parse this as a .cvsignore file */
-+#define MATCHFLG_PERDIR_MERGE (1<<9) /* content is inherited by subdirs */
-+#define MATCHFLG_INHERIT      (1<<10)/* content is inherited by subdirs */
++#define MATCHFLG_PERDIR_MERGE (1<<9) /* merge-file is searched per-dir */
++#define MATCHFLG_FINISH_SETUP (1<<10)/* per-dir merge file needs setup */
  struct exclude_struct {
        struct exclude_struct *next;
        char *pattern;
@@ -807,137 +930,297 @@ the -i option).
  };
  
  struct exclude_list_struct {
---- orig/rsync.yo      2004-08-03 15:34:32
-+++ rsync.yo   2004-08-06 17:05:29
-@@ -335,6 +335,7 @@ verb(
+--- orig/rsync.yo      2004-09-20 05:10:48
++++ rsync.yo   2004-08-13 00:43:31
+@@ -364,6 +364,7 @@ verb(
       --include=PATTERN       don't exclude files matching PATTERN
       --include-from=FILE     don't exclude patterns listed in FILE
       --files-from=FILE       read FILE for list of source-file names
-+ -E                          same as --exclude='. -pi /.rsync-excludes'
++ -E                          same as --exclude='. -p /.rsync-excludes'
   -0  --from0                 all file lists are delimited by nulls
       --version               print version number
       --daemon                run as an rsync daemon
-@@ -1086,6 +1087,11 @@ itemize(
-   then it is always considered an exclude pattern, even if specified as
-   part of an include option. The prefix is discarded before matching.
+@@ -1025,24 +1026,32 @@ The exclude and include patterns specifi
+ selection of which files to transfer and which files to skip.
+ Rsync builds an ordered list of include/exclude options as specified on
+-the command line. Rsync checks each file and directory 
+-name against each exclude/include pattern in turn. The first matching
++the command line.
++It can also be told to check for include/exclude options in each
++directory that rsync visits during the transfer (see the section on
++MERGED EXCLUDE FILES for the details on these per-directory exclude
++files).
++
++As the list of files/directories to transfer is built, rsync checks each
++name against every exclude/include pattern in turn. The first matching
+ pattern is acted on. If it is an exclude pattern, then that file is
+ skipped. If it is an include pattern then that filename is not
+ skipped. If no matching include/exclude pattern is found then the
+ filename is not skipped.
+-The filenames matched against the exclude/include patterns are relative
+-to the "root of the transfer".  If you think of the transfer as a
+-subtree of names that are being sent from sender to receiver, the root
+-is where the tree starts to be duplicated in the destination directory.
+-This root governs where patterns that start with a / match (see below).
++The global include/exclude rules are anchored at the "root of the
++transfer" (as opposed to per-directory rules, which are anchored at
++the merge-file's directory).  If you think of the transfer as a
++subtree of names that are being sent from sender to receiver, the
++transfer-root is where the tree starts to be duplicated in the
++destination directory.  This root governs where patterns that start
++with a / match (as described in the list on pattern forms below).
+ Because the matching is relative to the transfer-root, changing the
+ trailing slash on a source path or changing your use of the --relative
+ option affects the path you need to use in your matching (in addition to
+ changing how much of the file tree is duplicated on the destination
+-system).  The following examples demonstrate this.
++host).  The following examples demonstrate this.
+ Let's say that we want to match two source files, one with an absolute
+ path of "/home/me/foo/bar", and one with a path of "/home/you/bar/baz".
+@@ -1089,23 +1098,27 @@ because rsync did not descend through th
+ hierarchy.
+ Note also that the --include and --exclude options take one pattern
+-each. To add multiple patterns use the --include-from and
+---exclude-from options or multiple --include and --exclude options. 
++each. To add multiple patterns use the --include-from and --exclude-from
++options or multiple --include and --exclude options.
  
+-The patterns can take several forms. The rules are:
++The include/exclude patterns can take several forms. The rules are:
+ itemize(
+-  it() if the pattern starts with a / then it is matched against the
+-  start of the filename, otherwise it is matched against the end of
+-  the filename.
+-  This is the equivalent of a leading ^ in regular expressions.
+-  Thus "/foo" would match a file called "foo" at the transfer-root
+-  (see above for how this is different from the filesystem-root).
+-  On the other hand, "foo" would match any file called "foo"
++  it() if the pattern starts with a / then it is anchored to a
++  particular spot in the hierarchy of files, otherwise it is matched
++  against the end of the pathname.  This is similar to a leading ^ in
++  regular expressions.
++  Thus "/foo" would match a file called "foo" at either the "root of the
++  transfer" (for a global rule) or in the merge-file's directory (for a
++  per-directory rule).
++  An unqualified "foo" would match any file or directory named "foo"
+   anywhere in the tree because the algorithm is applied recursively from
++  the
+   top down; it behaves as if each path component gets a turn at being the
+-  end of the file name.
++  end of the file name.  Even the unanchored "sub/foo" would match at
++  any point in the hierarchy where a "foo" was found within a directory
++  named "sub".
+   it() if the pattern ends with a / then it will only match a
+   directory, not a file, link, or device.
+@@ -1118,22 +1131,31 @@ itemize(
+   single asterisk pattern "*" will stop at slashes.
+   it() if the pattern contains a / (not counting a trailing /) or a "**"
+-  then it is matched against the full filename, including any leading
+-  directory. If the pattern doesn't contain a / or a "**", then it is
++  then it is matched against the full pathname, including any leading
++  directories. If the pattern doesn't contain a / or a "**", then it is
+   matched only against the final component of the filename.  Again,
+   remember that the algorithm is applied recursively so "full filename" can
+   actually be any portion of a path below the starting directory.
+   it() if the pattern starts with "+ " (a plus followed by a space)
+   then it is always considered an include pattern, even if specified as
+-  part of an exclude option. The prefix is discarded before matching.
++  part of an exclude option. (The prefix is discarded before matching.)
+   it() if the pattern starts with "- " (a minus followed by a space)
+   then it is always considered an exclude pattern, even if specified as
+-  part of an include option. The prefix is discarded before matching.
++  part of an include option. (The prefix is discarded before matching.)
++
 +  it() if the pattern starts with ". " (a dot followed by a space) then its
-+  pattern is taken to be a merge file that is read in to supplement the
-+  current rules.  See the section on MERGING EXCLUDE FILES for more
++  pattern is taken to be a merge-file that is read in to supplement the
++  current rules.  See the section on MERGED EXCLUDE FILES for more
 +  information.
-+
    it() if the pattern is a single exclamation mark ! then the current
    include/exclude list is reset, removing all previously defined patterns.
- )
-@@ -1138,6 +1144,106 @@ itemize(
-   it would be excluded by the "*")
++  The "current" list is either the global list of rules (which are
++  specified via options) or a set of per-directory rules (which are
++  inherited in their own sub-list, so a subdirectory can use this to
++  clear out the parent's rules).
  )
  
-+manpagesection(MERGING EXCLUDE FILES)
+ The +/- rules are most useful in a list that was read from a file, allowing
+@@ -1180,8 +1202,160 @@ itemize(
+   it() --include "*/" --include "*.c" --exclude "*" would include all 
+   directories and C source files
+   it() --include "foo/" --include "foo/bar.c" --exclude "*" would include
+-  only foo/bar.c (the foo/ directory must be explicitly included or
+-  it would be excluded by the "*")
++  only the foo directory and foo/bar.c (the foo directory must be
++  explicitly included or it would be excluded by the "*")
++)
++
++manpagesection(MERGED EXCLUDE FILES)
 +
 +You can merge whole files into an exclude file by specifying a rule that
 +starts with a ". " (a dot followed by a space) and putting a filename in
-+place of the pattern.  The filename may be preceded by one or more options:
++place of the pattern.  There are two kinds of merged exclude files --
++single-instance and per-directory.  The choice is made via an option
++placed prior to the merge-file name:
 +
 +startdit()
 +
-+dit(bf(-i)) All subdirectories of the current directory inherit the rules we
-+read in from this file.  Only affects a per-directory merge file.  The
-+rules read in are prefixed to the inherited rules from a parent directory,
-+which gives the newest rules a higher priority than the inherited rules.
-+
-+Note also that you can eliminate all the inherited rules for a directory
-+and its subdirectories by putting the list-clearing token ! at the start of
-+a per-directory file.  This only clears the rules in the current sub-list,
-+not all the rules.
-+
 +dit(bf(-p)) Make the file a per-directory merge-file.  Rsync will scan
 +every directory that it traverses for the named file, merging its contents
-+when the file exists.  Without this option rsync just merges the rules into
-+the parent file, giving them the same attributes as the parent.
++when the file exists.  These exclude files must exist on the sending side
++because it is the sending side that is being scanned for available files
++to send.  The files may also need to be transferred to the receiving side
++if you want them to affect what files don't get deleted (see PER-DIRECTORY
++EXCLUDES AND DELETE below).
 +
 +dit(bf(--)) End the scanning of options.  Useful if you want to specify a
 +filename that begins with a dash.
 +
 +enddit()
 +
-+Here's an example exclude file (which you'd specify via the normal
-+--exclude-from=FILE option):
++Per-directory rules are inherited in all subdirectories of the directory
++where the merge-file was found.  Each subdirectory's rules are prefixed
++to the inherited rules from the parent directories, which gives the
++newest rules a higher priority than the inherited rules.  The entire set
++of per-dir rules is grouped together in the spot where the merge-file was
++specified, so it is possible to override per-dir rules via a rule that
++got specified earlier in the list of global rules.
++
++If you don't want a per-dir rule to be inherited, anchor it with a leading
++slash.  Anchored rules in a per-directory merge-file are relative to the
++merge-file's directory, so a rule "/foo" would only exclude the file "foo"
++in the directory where the per-dir exclude file was found.
++
++Here's an example exclude file which you'd specify via the normal
++--exclude-from=FILE option:
 +
 +verb(
 +    . /home/user/.global_excludes
 +    *.gz
-+    . -pi .excl
++    . -p .excl
 +    + *.[ch]
 +    *.o
 +)
 +
 +This will merge the contents of the /home/user/.global_excludes file at the
 +start of the list and also turns the ".excl" filename into a per-directory
-+exclude file.  The rules read in from the .global_excludes file affect all
-+the directories because it is is being merged into an --exclude-from
-+option.  The rules merged from each directory's .excl file are inherited
-+by each directory's subdirectories because the -i option was specified
-+(without -i the rules would only affect the directory where they were read
-+in).  All the merged rules default to being exclude rules because an
-+exclude statement was used to specify them.
-+
-+Note also that the parsing of any merge-file named ".cvsignore" is always
-+done in a CVS-compatible manner, even if -C wasn't specified.  This means
-+that its rules are always excludes (even if an include option specified the
-+file), tokens are split on whitespace, the rules are never inherited, and
-+no special characters are honored (e.g. no comments, no "!", etc.).
++exclude file.  All the merged rules default to being exclude rules because
++an exclude statement was used to specify them.  Rules read in from the
++.global_excludes file are anchored just like all other global rules.
++
++If a per-directory merge-file is specified with a path that is a parent
++directory of the first transfer directory, rsync will scan all the parent
++dirs from that starting point to the transfer directory for the indicated
++per-directory file.  For instance, the -E option is an abbreviation for
++this command:
++
++verb(
++  --exclude='. -p /.rsync-excludes'
++)
++
++That exclude tells rsync to scan for the file .rsync-excludes in all
++directories from the root down through the source of the transfer.  (For
++an rsync daemon, the "root dir" is always the module's "path" setting.)
++
++Some examples of this pre-scanning for per-directory files:
++
++verb(
++  rsync -avE /src/path/ /dest/dir
++  rsync -av --exclude='. -p ../../.rsync-excludes' /src/path/ /dest/dir
++  rsync -av --exclude='. -p .rsync-excludes' /src/path/ /dest/dir
++)
++
++The first two commands above will look for ".rsync-excludes" in "/" and
++"/src" before the normal scan begins looking for the file in "/src/path"
++and its subdirectories.  The last command avoids the parent-dir scan
++and only looks for the ".rsync-excludes" files in each directory that is
++a part of the transfer.
++
++Finally, note that the parsing of any merge-file named ".cvsignore" is
++always done in a CVS-compatible manner, even if -C wasn't specified.  This
++means that its rules are always excludes (even if an include option
++specified the file), patterns are split on whitespace, the rules are never
++inherited, and no special characters are honored (e.g. no comments, no "!",
++etc.).
 +
 +Additionally, you can affect where the --cvs-exclude (-C) option's
 +inclusion of the per-directory .cvsignore file gets placed into your rules
 +by adding your own explicit per-directory merge rule for ".cvsignore".
-+Without this rsync would add its this rule at the end of all your other
++Without this, rsync would add its this rule at the end of all your other
 +rules (giving it a lower priority than your command-line rules).  For
-+example, specifying this:
++example:
 +
 +verb(
 +  rsync -avC --exclude='. -p .cvsignore' --exclude-from=foo a/ b
 +)
 +
-+will merge all the per-directory .cvsignore rules at the start of your list
-+rather than at the end.  This allows their dir-specific rules to supersede
-+your rules instead of being subservient to them.  (The global rules taken
-+from the $HOME/.cvsignore file and from $CVSIGNORE are not repositioned by
-+this.)
++The above will merge all the per-directory .cvsignore rules at the start of
++your list rather than at the end.  This allows their dir-specific rules to
++supersede your rules instead of being subservient to them.  (The global
++rules taken from the $HOME/.cvsignore file and from $CVSIGNORE are not
++repositioned by this.)
 +
-+If a per-directory merge file is specified with a path that is a parent
-+directory of the first transfer directory, rsync will scan all the parent
-+dirs from that starting point to the transfer directory for the indicated
-+per-directory file.  For instance, the -E option is an abbreviation for
-+this command:
++manpagesection(PER-DIRECTORY EXCLUDES AND DELETE)
++
++Without a delete option, per-directory excludes are only relevant on the
++sending side, so you can feel free to exclude the merge files themselves
++without affecting the transfer:
 +
 +verb(
-+  --exclude='. -pi /.rsync-excludes'
++  rsync -av --exclude='. -p .excl' --exclude=.excl host:src/dir /dest
 +)
 +
-+That exclude tells rsync to scan for the file .rsync-excludes in all
-+directories from the root up through the source of the transfer.  For an
-+rsync daemon, the root dir is always the module's "path" setting, not the
-+root of the filesystem (unless the two are the same).
++However, if you want to do a delete on the receiving side AND you want some
++files to be excluded from being deleted, you'll need to be sure that the
++receiving side knows what files to exclude.  The easiest way is to include
++the per-directory merge files in the transfer and use --delete-after
++because this ensures that the receiving side gets all the same exclude
++rules as the sending side before it tries to delete anything:
 +
-+Some examples of this pre-scanning for per-directory files:
++verb(
++  rsync -avE --delete-after host:src/dir /dest
++)
++
++However, if you the merge files are not a part of the transfer, you'll need
++to either use a global exclude rule (i.e. specified on the command line),
++or you'll need to maintain your own per-directory merge files on the
++receiving side.  An example of the first is this (assume that the remote
++.ctrl files exclude themselves):
 +
 +verb(
-+  rsync -avE /src/path/ /dest/dir
-+  rsync -av --exclude='. -p ../../.rsync-excludes' /src/path/ /dest/dir
-+  rsync -av --exclude='. -pi .rsync-excludes' /src/path/ /dest/dir
++  rsync -av --exclude='. -p .ctrl' --exclude-from=/my/extra.rules
++    --delete host:src/dir /dest
 +)
 +
-+The first two commands above will look for .rsync-excludes in "/" and
-+"/src" before the normal scan begins looking for the file in "/src/path"
-+and its subdirectories.  The last command above just starts scanning
-+from "/src/path".
++In the above example the extra.rules file can affect both sides of the
++transfer, but the rules are subservient to the rules merged from the .ctrl
++files because they were specified after the per-directory merge rule.
 +
- manpagesection(BATCH MODE)
++In the final example, the remote side is excluding the .rsync-excludes
++files from the transfer, but we want to use our own .rsync-excludes files
++to control what gets deleted on the receiving side.  To do this we must
++specifically exclude the per-directory merge files (so that they don't get
++deleted) and then put rules into the local files to control what else
++should not get deleted.  Like this:
++
++verb(
++  rsync -avE --exclude=.rsync-excludes --delete host:src/dir /dest
+ )
  
- bf(Note:) Batch mode should be considered experimental in this version
+ manpagesection(BATCH MODE)
 --- orig/testsuite/exclude.test        2004-05-29 21:25:45
-+++ testsuite/exclude.test     2004-08-06 09:02:29
++++ testsuite/exclude.test     2004-08-08 06:35:15
 @@ -23,19 +23,47 @@ export HOME CVSIGNORE
  makepath "$fromdir/foo/down/to/you"
  makepath "$fromdir/bar/down/to/foo/too"
@@ -957,7 +1240,7 @@ the -i option).
 +EOF
 +cat >"$fromdir/bar/.excl" <<EOF
 +home-cvs-exclude
-+. -pi .excl2
++. -p .excl2
 ++ to
 +EOF
  echo cvsout >"$fromdir/bar/down/to/home-cvs-exclude"
@@ -1006,7 +1289,7 @@ the -i option).
 +# Now, test if rsync excludes the same files, this time with a merge-exclude
 +# file.
 +
-+checkit "$RSYNC -avv --exclude='. -pi .excl' --exclude-from=\"$excl\" \
++checkit "$RSYNC -avv --exclude='. -p .excl' --exclude-from=\"$excl\" \
 +    --delete-excluded \"$fromdir/\" \"$todir/\"" "$chkdir" "$todir"
 +
  # The script would have aborted on error, so getting here means we've won.