Leave normalization of patterns to util._matcher
authorAlexis S. L. Carvalho <alexis@cecm.usp.br>
Sat, 10 Mar 2007 23:00:46 -0300
changeset 4186 08d31e43592a
parent 4185 51ee2868a571
child 4187 01c4ea5e788c
Leave normalization of patterns to util._matcher Passing [] to util.cmdmatcher accidentally fixes walking of files with "\n" in the name.
mercurial/cmdutil.py
tests/test-issue352
tests/test-issue352.out
--- a/mercurial/cmdutil.py	Sat Mar 10 23:00:45 2007 -0300
+++ b/mercurial/cmdutil.py	Sat Mar 10 23:00:46 2007 -0300
@@ -129,13 +129,7 @@
 
 def matchpats(repo, pats=[], opts={}, head='', globbed=False):
     cwd = repo.getcwd()
-    if not pats and cwd:
-        opts['include'] = [os.path.join(cwd, i)
-                           for i in opts.get('include', [])]
-        opts['exclude'] = [os.path.join(cwd, x)
-                           for x in opts.get('exclude', [])]
-        cwd = ''
-    return util.cmdmatcher(repo.root, cwd, pats or ['.'], opts.get('include'),
+    return util.cmdmatcher(repo.root, cwd, pats or [], opts.get('include'),
                            opts.get('exclude'), head, globbed=globbed)
 
 def walk(repo, pats=[], opts={}, node=None, head='', badmatch=None,
--- a/tests/test-issue352	Sat Mar 10 23:00:45 2007 -0300
+++ b/tests/test-issue352	Sat Mar 10 23:00:46 2007 -0300
@@ -6,16 +6,17 @@
 
 A=`echo -e -n 'he\rllo'`
 
-echo foo > "hell
-o"
 echo foo > "$A"
 hg add
 hg ci -A -m m
 rm "$A"
-ls
+
+echo foo > "hell
+o"
 hg add
-# BUG ? we don't walk on filenames with '\n' (regexp related) ?
-hg debugwalk
 hg ci -A -m m
 
+echo foo > "$A"
+hg debugwalk
+
 exit 0
--- a/tests/test-issue352.out	Sat Mar 10 23:00:45 2007 -0300
+++ b/tests/test-issue352.out	Sat Mar 10 23:00:46 2007 -0300
@@ -2,6 +2,13 @@
 abort: '\n' and '\r' disallowed in filenames
 adding he
llo
 abort: '\n' and '\r' disallowed in filenames
-hell
+adding hell
+o
+abort: '\n' and '\r' disallowed in filenames
+adding hell
 o
-nothing changed
+abort: '\n' and '\r' disallowed in filenames
+f  he
llo  he
llo
+f  hell
+o  hell
+o