This patch causes the --cvs-exclude option to prefix the names listed in each dir's CVS/Entries file as per-dir includes before the dir's list of excludes taken from the .cvsignore file. To use this patch, run these commands for a successful build: patch -p1 head = lp->tail = lp->parent_dirscan_head = NULL; + if (mflags & MATCHFLG_CVS_IGNORE) + cp = "CVS"; if (asprintf(&lp->debug_type, " [per-dir %s]", cp) < 0) out_of_memory("add_rule"); rule->u.mergelist = lp; @@ -526,6 +528,14 @@ void *push_local_filters(const char *dir, unsigned int dirlen) set_filter_dir(dir, dirlen); } + if (ex->match_flags & MATCHFLG_CVS_IGNORE + && strlcpy(dirbuf + dirbuf_len, "CVS/Entries", + MAXPATHLEN - dirbuf_len) < MAXPATHLEN - dirbuf_len) { + /* Start by adding include rules for all the names in CVS/Entries. */ + parse_filter_file(lp, dirbuf, + MATCHFLG_NO_PREFIXES | MATCHFLG_INCLUDE, + XFLG_CVS_ENTRIES); + } if (strlcpy(dirbuf + dirbuf_len, ex->pattern, MAXPATHLEN - dirbuf_len) < MAXPATHLEN - dirbuf_len) { parse_filter_file(lp, dirbuf, ex, @@ -1151,6 +1161,7 @@ void parse_filter_file(filter_rule_list *listp, const char *fname, const filter_ char line[BIGPATHBUFLEN]; char *eob = line + sizeof line - 1; BOOL word_split = (template->rflags & FILTRULE_WORD_SPLIT) != 0; + int slash_parse = xflags & XFLG_CVS_ENTRIES ? 1 : 0; if (!fname || !*fname) return; @@ -1197,6 +1208,24 @@ void parse_filter_file(filter_rule_list *listp, const char *fname, const filter_ } break; } + switch (slash_parse) { /* CVS/Entries parsing: */ + case 1: /* Ignore starting chars until first slash. */ + if (ch == '/') + slash_parse = 2; + continue; + case 2: /* Name ends at 2nd slash on the line. */ + if (ch == '/') { + slash_parse = 3; + continue; + } + break; + case 3: /* Ignore trailing chars until EOL. */ + if (ch == '\n' || ch == '\r') { + slash_parse = 1; + goto end_the_line; + } + continue; + } if (word_split && isspace(ch)) break; if (eol_nulls? !ch : (ch == '\n' || ch == '\r')) @@ -1206,13 +1235,14 @@ void parse_filter_file(filter_rule_list *listp, const char *fname, const filter_ else overflow = 1; } + end_the_line: if (overflow) { rprintf(FERROR, "discarding over-long filter: %s...\n", line); s = line; } *s = '\0'; /* Skip an empty token and (when line parsing) comments. */ - if (*line && (word_split || (*line != ';' && *line != '#'))) + if (*line && (word_split || slash_parse || (*line != ';' && *line != '#'))) parse_filter_str(listp, line, template, xflags); if (ch == EOF) break; diff --git a/rsync.h b/rsync.h --- a/rsync.h +++ b/rsync.h @@ -157,6 +157,7 @@ #define XFLG_ANCHORED2ABS (1<<2) /* leading slash indicates absolute */ #define XFLG_ABS_IF_SLASH (1<<3) /* leading or interior slash is absolute */ #define XFLG_DIR2WILD3 (1<<4) /* dir/ match gets trailing *** added */ +#define XFLG_CVS_ENTRIES (1<<5) #define ATTRS_REPORT (1<<0) #define ATTRS_SKIP_MTIME (1<<1) diff --git a/testsuite/exclude.test b/testsuite/exclude.test --- a/testsuite/exclude.test +++ b/testsuite/exclude.test @@ -19,6 +19,7 @@ export CVSIGNORE makepath "$fromdir/foo/down/to/you" makepath "$fromdir/foo/sub" +makepath "$fromdir/bar/down/to/CVS" makepath "$fromdir/bar/down/to/foo/too" makepath "$fromdir/bar/down/to/bar/baz" makepath "$fromdir/mid/for/foo/and/that/is/who" @@ -57,6 +58,9 @@ echo cvsout >"$fromdir/bar/down/to/foo/file1.bak" echo gone >"$fromdir/bar/down/to/foo/file3" echo lost >"$fromdir/bar/down/to/foo/file4" echo weird >"$fromdir/bar/down/to/foo/+ file3" +echo cvsin >"$fromdir/bar/down/to/not.junk" +echo cvsout >"$fromdir/bar/down/to/not.good" +echo cvsout >"$fromdir/bar/down/to/D" echo cvsout-but-filtin >"$fromdir/bar/down/to/foo/file4.junk" echo smashed >"$fromdir/bar/down/to/foo/to" cat >"$fromdir/bar/down/to/bar/.filt2" <"$excl" <"$scratchdir/.cvsignore" <"$fromdir/bar/down/to/CVS/Entries" <"$fromdir/bar/down/to/.cvsignore" <