From 01099707dbc1b7c774253ac0cd278c2c09272a30 Mon Sep 17 00:00:00 2001 From: "Gregory P. Smith" Date: Sun, 16 Aug 2009 18:58:46 +0000 Subject: [PATCH] Merged revisions 74475 via svnmerge from svn+ssh://pythondev@svn.python.org/python/trunk ........ r74475 | gregory.p.smith | 2009-08-16 11:52:58 -0700 (Sun, 16 Aug 2009) | 2 lines Issue 6665: Fix fnmatch to properly match filenames with newlines in them. ........ --- Lib/fnmatch.py | 2 +- Lib/test/test_fnmatch.py | 11 ++++++++++- Misc/NEWS | 2 ++ 3 files changed, 13 insertions(+), 2 deletions(-) diff --git a/Lib/fnmatch.py b/Lib/fnmatch.py index 0397549d4dd..30c0a922f22 100644 --- a/Lib/fnmatch.py +++ b/Lib/fnmatch.py @@ -113,4 +113,4 @@ def translate(pat): res = '%s[%s]' % (res, stuff) else: res = res + re.escape(c) - return res + "$" + return res + '\Z(?ms)' diff --git a/Lib/test/test_fnmatch.py b/Lib/test/test_fnmatch.py index 2f701caccaa..506c67999c5 100644 --- a/Lib/test/test_fnmatch.py +++ b/Lib/test/test_fnmatch.py @@ -32,11 +32,18 @@ class FnmatchTestCase(unittest.TestCase): check('a', 'b', 0) # these test that '\' is handled correctly in character sets; - # see SF bug #??? + # see SF bug #409651 check('\\', r'[\]') check('a', r'[!\]') check('\\', r'[!\]', 0) + # test that filenames with newlines in them are handled correctly. + # http://bugs.python.org/issue6665 + check('foo\nbar', 'foo*') + check('foo\nbar\n', 'foo*') + check('\nfoo', 'foo*', False) + check('\n', '*') + def test_mix_bytes_str(self): self.assertRaises(TypeError, fnmatch, 'test', b'*') self.assertRaises(TypeError, fnmatch, b'test', '*') @@ -46,6 +53,8 @@ class FnmatchTestCase(unittest.TestCase): def test_bytes(self): self.check_match(b'test', b'te*') self.check_match(b'test\xff', b'te*\xff') + self.check_match(b'foo\nbar', b'foo*') + def test_main(): support.run_unittest(FnmatchTestCase) diff --git a/Misc/NEWS b/Misc/NEWS index c861b45638a..42357809182 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -65,6 +65,8 @@ C-API Library ------- +- Issue #6665: Fix fnmatch to properly match filenames with newlines in them. + - Issue #1135: Add the XView and YView mix-ins to avoid duplicating the xview* and yview* methods.