Need to make sure that the destination file doesn't exist before we
[rsync/rsync-patches.git] / filter.diff
index 28a6dec..ffaf81c 100644 (file)
@@ -22,39 +22,48 @@ this:
   . -p .excl2
   . .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).  The file ".excl3" would just be
-read in from the current dir (because it was not specified with the -p
-option).
+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/clientserver.c        2004-08-02 02:29:16
-+++ clientserver.c     2004-08-09 02:25:28
-@@ -48,9 +48,10 @@ extern int no_detach;
++++ 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;
-+extern char dirbuf[MAXPATHLEN];
-+extern unsigned int dirbuf_offset;
  
  char *auth_user;
  
-@@ -300,26 +301,27 @@ static int rsync_module(int f_in, int f_
++/* 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)
-+              set_current_subdir("", 0);
-+      else
-+              set_current_subdir(lp_path(i), strlen(lp_path(i)));
++      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,
@@ -73,31 +82,24 @@ option).
  
        p = lp_exclude(i);
 -      add_exclude(&server_exclude_list, p, XFLG_WORD_SPLIT);
-+      add_exclude(&server_exclude_list, p, XFLG_WORD_SPLIT | XFLG_ABS_PATH);
+-
 -      exclude_path_prefix = NULL;
-+      set_current_subdir("", 0);
++      add_exclude(&server_exclude_list, p, XFLG_WORD_SPLIT | XFLG_ABS_PATH);
  
        log_init();
  
---- orig/exclude.c     2004-08-05 23:16:37
-+++ exclude.c  2004-08-09 03:19:40
-@@ -27,16 +27,80 @@
- #include "rsync.h"
- extern int verbose;
-+extern int am_sender;
+--- 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 " };
@@ -111,19 +113,14 @@ option).
 +
 +/* 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_offset contains the
-+ * length of this path prefix (i.e. it is an offset where you can copy a
-+ * filename to create a pathname that can be opened for reading/writing.
-+ *
-+ * The path is normally relative (e.g. "sub/dir/foo"), but it is set to an
-+ * absolute path when the push code is working on a parent-dir scan of dirs
-+ * that might be higher than the root of the transfer.  In that case, the
-+ * path is absolute, and any newly-created per-dir merge files will (at
-+ * least temporarily) get an absolute file name so that we know at what
-+ * point in the hierarchy it first makes an appearance. */
-+static char dirbuf[MAXPATHLEN];
-+static unsigned int dirbuf_offset = 0;
-+static BOOL parent_dirscan = 0;
++ * 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
@@ -165,7 +162,7 @@ option).
  
  /** Build an exclude structure given an exclude pattern. */
  static void make_exclude(struct exclude_list_struct *listp, const char *pat,
-@@ -46,23 +110,56 @@ static void make_exclude(struct exclude_
+@@ -46,23 +102,50 @@ static void make_exclude(struct exclude_
        const char *cp;
        unsigned int ex_len;
  
@@ -209,27 +206,20 @@ option).
 +              if (*pat != '/') {
 +                      mflags &= ~MATCHFLG_ABS_PATH;
 +                      ex_len = 0;
-+              } else {
-+                      ex_len = *dirbuf == '/' ? dirbuf_offset - 1
-+                             : curr_dir_len + dirbuf_offset - !curr_dir[1];
-+              }
++              } 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)
+       if (ex_len)
 -              memcpy(ret->pattern, exclude_path_prefix, ex_len);
-+      if (ex_len) {
-+              if (*dirbuf == '/')
-+                      strlcpy(ret->pattern, dirbuf, ex_len + 1);
-+              else
-+                      pathjoin(ret->pattern, ex_len + 1, curr_dir, dirbuf);
-+      }
++              memcpy(ret->pattern, dirbuf + module_dirlen, ex_len);
        strlcpy(ret->pattern + ex_len, pat, pat_len + 1);
        pat_len += ex_len;
  
-@@ -81,14 +178,40 @@ static void make_exclude(struct exclude_
+@@ -81,14 +164,40 @@ static void make_exclude(struct exclude_
                mflags |= MATCHFLG_DIRECTORY;
        }
  
@@ -274,7 +264,7 @@ option).
                listp->tail->next = ret;
                listp->tail = ret;
        }
-@@ -96,22 +219,265 @@ static void make_exclude(struct exclude_
+@@ -96,22 +205,267 @@ static void make_exclude(struct exclude_
  
  static void free_exclude(struct exclude_struct *ex)
  {
@@ -309,18 +299,19 @@ option).
  }
  
 +/* 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 non-zero;
++ * 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 not null terminated and the length
-+ * value is contained therein (and will be updated with the new length). We
-+ * always return a name that is null terminated. */
-+static char *parse_merge_name(const char *merge_file, unsigned int *len_ptr)
++ * 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 == '/' ? 0 : curr_dir_len + 1;
 +      if (!parent_dirscan && *merge_file != '/') {
 +              /* Return the name unchanged it doesn't have any slashes. */
 +              if (len_ptr) {
@@ -336,80 +327,83 @@ option).
 +
 +      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;
 +              }
-+              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;
 +}
 +
-+/* Sets the dirbuf and dirbuf_offset values */
-+void set_current_subdir(const char *dir, unsigned int dirlen)
++/* Sets the dirbuf and dirbuf_len values. */
++void set_excludes_dir(const char *dir, unsigned int dirlen)
 +{
-+      memcpy(dirbuf, dir, dirlen);
-+      dirbuf[dirlen] = '\0';
-+      dirbuf_offset = clean_fname(dirbuf);
-+      if (dirbuf_offset == 1 && *dirbuf == '.')
-+              dirbuf_offset = 0;
-+      else
-+              dirbuf[dirbuf_offset++] = '/';
-+      dirbuf[dirbuf_offset] = '\0';
++      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.
++/* 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 void setup_merge_file(struct exclude_struct *ex,
-+                           struct exclude_list_struct *lp, int flags,
-+                           const char *dir, unsigned int dirlen)
++ * 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, *pat = ex->pattern;
 +      unsigned int len;
 +
-+      if (!(x = parse_merge_name(pat, NULL)) || *x != '/')
-+              return;
++      if (!(x = parse_merge_name(pat, NULL, 0)) || *x != '/')
++              return 0;
 +
 +      y = strrchr(x, '/');
 +      *y = '\0';
 +      ex->pattern = strdup(y+1);
-+      if (*dirbuf != '/') {
-+              pathjoin(dirbuf, MAXPATHLEN, sanitize_paths ? "/" : curr_dir,
-+                       dir);
-+      }
 +      if (!*x)
 +              x = "/";
 +      if (*x == '/')
@@ -417,25 +411,22 @@ option).
 +      else
 +              pathjoin(buf, MAXPATHLEN, dirbuf, x);
 +
-+      len = clean_fname(buf);
++      len = clean_fname(buf, 1);
 +      if (len != 1 && len < MAXPATHLEN-1) {
 +              buf[len++] = '/';
 +              buf[len] = '\0';
 +      }
-+      x = buf;
-+      if (sanitize_paths)
-+              x += strlen(lp_path(module_id));
 +      /* This ensures that the specified dir is a parent of the transfer. */
-+      for (y = dirbuf; *x && *x == *y; x++, y++) {}
++      for (x = buf, y = dirbuf; *x && *x == *y; x++, y++) {}
 +      if (*x)
 +              y += strlen(y); /* nope -- skip the scan */
 +
-+      parent_dirscan = 1;
++      parent_dirscan = True;
 +      while (*y) {
 +              char save[MAXPATHLEN];
 +              strlcpy(save, y, MAXPATHLEN);
 +              *y = '\0';
-+              dirbuf_offset = y - dirbuf;
++              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)
@@ -444,22 +435,22 @@ option).
 +              strlcpy(y, save, MAXPATHLEN);
 +              while ((*x++ = *y++) != '/') {}
 +      }
-+      parent_dirscan = 0;
++      parent_dirscan = False;
 +      free(pat);
-+      set_current_subdir(dir, dirlen);
++      return 1;
 +}
 +
 +/* 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 (with a mandatory trailing slash).  We copy it into
-+ * dirbuf so that the routines this calls can append a file name. */
++ * 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;
 +
-+      set_current_subdir(dir, dirlen);
++      set_excludes_dir(dir, dirlen);
 +
 +      if (!(push = new_array(struct mergelist_save_struct, 1)))
 +              out_of_memory("push_local_excludes");
@@ -497,11 +488,12 @@ option).
 +
 +              if (ex->match_flags & MATCHFLG_FINISH_SETUP) {
 +                      ex->match_flags &= ~MATCHFLG_FINISH_SETUP;
-+                      setup_merge_file(ex, lp, flags, dir, dirlen);
++                      if (setup_merge_file(ex, lp, flags))
++                              set_excludes_dir(dir, dirlen);
 +              }
 +
-+              if (strlcpy(dirbuf + dirbuf_offset, ex->pattern,
-+                  MAXPATHLEN - dirbuf_offset) < MAXPATHLEN - dirbuf_offset)
++              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;
@@ -509,7 +501,7 @@ option).
 +                          "cannot add local excludes in long-named directory %s\n",
 +                          full_fname(dirbuf));
 +              }
-+              dirbuf[dirbuf_offset] = '\0';
++              dirbuf[dirbuf_len] = '\0';
 +      }
 +
 +      return (void*)push;
@@ -546,7 +538,7 @@ option).
  static int check_one_exclude(char *name, struct exclude_struct *ex,
                               int name_is_dir)
  {
-@@ -122,7 +488,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. */
@@ -555,17 +547,16 @@ option).
                if ((p = strrchr(name,'/')) != NULL)
                        name = p+1;
        }
-@@ -133,7 +499,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 +515,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. */
@@ -577,7 +568,7 @@ option).
                        for (p = name + strlen(name) - 1; p >= name; p--) {
                                if (*p == '/' && !--cnt)
                                        break;
-@@ -221,6 +588,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) {
@@ -591,7 +582,7 @@ option).
                if (check_one_exclude(name, ent, name_is_dir)) {
                        report_exclude_result(name, ent, name_is_dir,
                                              listp->debug_type);
-@@ -253,11 +627,36 @@ static const char *get_exclude_tok(const
+@@ -253,11 +615,36 @@ static const char *get_exclude_tok(const
                p = (const char *)s;
        }
  
@@ -630,7 +621,7 @@ option).
                s += 2;
        } else if (xflags & XFLG_DEF_INCLUDE)
                mflags |= MATCHFLG_INCLUDE;
-@@ -273,6 +672,8 @@ static const char *get_exclude_tok(const
+@@ -273,6 +660,8 @@ static const char *get_exclude_tok(const
  
        if (*p == '!' && len == 1 && !(xflags & XFLG_WORDS_ONLY))
                mflags |= MATCHFLG_CLEAR_LIST;
@@ -639,7 +630,16 @@ option).
  
        *len_ptr = len;
        *flag_ptr = mflags;
-@@ -292,9 +693,15 @@ void add_exclude(struct exclude_list_str
+@@ -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;
+       if (!pattern)
+               return;
+@@ -292,9 +681,15 @@ void add_exclude(struct exclude_list_str
        cp = pattern;
        pat_len = 0;
        while (1) {
@@ -655,7 +655,7 @@ option).
  
                if (mflags & MATCHFLG_CLEAR_LIST) {
                        if (verbose > 2) {
-@@ -306,13 +713,23 @@ void add_exclude(struct exclude_list_str
+@@ -306,13 +701,24 @@ void add_exclude(struct exclude_list_str
                        continue;
                }
  
@@ -666,17 +666,18 @@ 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 (parent_dirscan) {
-+                                      if (!(cp = parse_merge_name(cp, &pat_len)))
++                                      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;
 +                      }
                }
@@ -685,7 +686,7 @@ option).
        }
  }
  
-@@ -321,7 +738,7 @@ void add_exclude_file(struct exclude_lis
+@@ -321,7 +727,7 @@ void add_exclude_file(struct exclude_lis
                      int xflags)
  {
        FILE *fp;
@@ -694,19 +695,20 @@ option).
        char *eob = line + sizeof line - 1;
        int word_split = xflags & XFLG_WORD_SPLIT;
  
-@@ -343,6 +760,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 +824,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);
@@ -729,7 +731,7 @@ option).
                        write_int(f, l + 2);
                        write_buf(f, "- ", 2);
                } else
-@@ -443,6 +879,7 @@ void add_cvs_excludes(void)
+@@ -443,6 +869,7 @@ void add_cvs_excludes(void)
        char fname[MAXPATHLEN];
        char *p;
  
@@ -737,9 +739,9 @@ 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-09 02:45:42
-@@ -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;
  
@@ -751,7 +753,7 @@ 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;
@@ -759,7 +761,7 @@ 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)
  {
@@ -768,7 +770,7 @@ 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
@@ -780,7 +782,7 @@ 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)) {
@@ -796,7 +798,7 @@ 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;
@@ -804,7 +806,7 @@ 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++;
        }
  
@@ -824,7 +826,7 @@ 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);
        }
  
@@ -833,44 +835,43 @@ 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;
-+      BOOL did_first_push = 0;
++      BOOL need_first_push = True;
        int64 start_write;
        int use_ff_fd = 0;
  
-@@ -1067,6 +1045,10 @@ 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) {
 +                              push_local_excludes(curr_dir, curr_dir_len);
-+                              did_first_push = 1;
++                              need_first_push = False;
 +                      }
                }
        }
  
-@@ -1097,6 +1079,16 @@ struct file_list *send_file_list(int f, 
+@@ -1121,6 +1103,15 @@ struct file_list *send_file_list(int f, 
                        }
                }
  
-+              if (!did_first_push) {
++              if (need_first_push) {
 +                      if ((p = strrchr(fname, '/')) != NULL) {
-+                              dir = fname;
-+                              p++;
-+                      } else
-+                              dir = p = "";
-+                      push_local_excludes(dir, p - dir);
-+                      did_first_push = 1;
++                              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-09 03:33:31
+--- 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");
@@ -887,7 +888,7 @@ 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 +591,11 @@ int parse_arguments(int *argc, const cha
+@@ -585,6 +587,11 @@ int parse_arguments(int *argc, const cha
                        am_sender = 1;
                        break;
  
@@ -929,9 +930,9 @@ option).
  };
  
  struct exclude_list_struct {
---- orig/rsync.yo      2004-08-03 15:34:32
-+++ rsync.yo   2004-08-09 03:10: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
@@ -939,55 +940,170 @@ option).
   -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,104 @@ 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(-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.)
-+
-+Rules are inherited in all subdirectories of the directory where the
-+per-dir merge file was found.  Each subdirectory's rules are prefixed to
-+the inherited rules from a parent directory, which gives the newest rules a
-+higher priority than the inherited rules.  If you don't want a rule to be
-+inherited, anchor it with a leading slash.  Anchored rules in a
-+per-directory merge file are relative to the current directory, so a rule
-+"/foo" would only exclude the file "foo" in the current directory.
-+
-+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.
++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
@@ -1000,32 +1116,10 @@ option).
 +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.  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.).
++an exclude statement was used to specify them.  Rules read in from the
++.global_excludes file are anchored just like all other global rules.
 +
-+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
-+rules (giving it a lower priority than your command-line rules).  For
-+example, specifying this:
-+
-+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.)
-+
-+If a per-directory merge file is specified with a path that is a parent
++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
@@ -1036,9 +1130,8 @@ option).
 +)
 +
 +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).
++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:
 +
@@ -1048,14 +1141,84 @@ option).
 +  rsync -av --exclude='. -p .rsync-excludes' /src/path/ /dest/dir
 +)
 +
-+The first two commands above will look for .rsync-excludes in "/" and
++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".
++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.
 +
- manpagesection(BATCH MODE)
++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
++rules (giving it a lower priority than your command-line rules).  For
++example:
++
++verb(
++  rsync -avC --exclude='. -p .cvsignore' --exclude-from=foo a/ b
++)
++
++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.)
++
++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(
++  rsync -av --exclude='. -p .excl' --exclude=.excl host:src/dir /dest
++)
++
++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:
++
++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 -av --exclude='. -p .ctrl' --exclude-from=/my/extra.rules
++    --delete host:src/dir /dest
++)
++
++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.
++
++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-08 06:35:15
 @@ -23,19 +23,47 @@ export HOME CVSIGNORE