[Python-checkins] r76044 - in python/branches/release26-maint/Lib/distutils: filelist.py tests/test_filelist.py

[tarek.ziade]

Author: tarek.ziade
Date: Mon Nov  2 00:04:26 2009
New Revision: 76044

Log:
reapplied r74493 (after #6665 fix has been backported)

Modified:
   python/branches/release26-maint/Lib/distutils/filelist.py
   python/branches/release26-maint/Lib/distutils/tests/test_filelist.py

Modified: python/branches/release26-maint/Lib/distutils/filelist.py
==============================================================================
--- python/branches/release26-maint/Lib/distutils/filelist.py	(original)
+++ python/branches/release26-maint/Lib/distutils/filelist.py	Mon Nov  2 00:04:26 2009
@@ -344,7 +344,9 @@
         pattern_re = ''
 
     if prefix is not None:
-        prefix_re = (glob_to_re(prefix))[0:-1] # ditch trailing $
+        # ditch end of pattern character
+        empty_pattern = glob_to_re('')
+        prefix_re = (glob_to_re(prefix))[:-len(empty_pattern)]
         pattern_re = "^" + os.path.join(prefix_re, ".*" + pattern_re)
     else:                               # no prefix -- respect anchor flag
         if anchor:

Modified: python/branches/release26-maint/Lib/distutils/tests/test_filelist.py
==============================================================================
--- python/branches/release26-maint/Lib/distutils/tests/test_filelist.py	(original)
+++ python/branches/release26-maint/Lib/distutils/tests/test_filelist.py	Mon Nov  2 00:04:26 2009
@@ -6,15 +6,15 @@
 
     def test_glob_to_re(self):
         # simple cases
-        self.assertEquals(glob_to_re('foo*'), 'foo[^/]*$')
-        self.assertEquals(glob_to_re('foo?'), 'foo[^/]$')
-        self.assertEquals(glob_to_re('foo??'), 'foo[^/][^/]$')
+        self.assertEquals(glob_to_re('foo*'), 'foo[^/]*\\Z(?ms)')
+        self.assertEquals(glob_to_re('foo?'), 'foo[^/]\\Z(?ms)')
+        self.assertEquals(glob_to_re('foo??'), 'foo[^/][^/]\\Z(?ms)')
 
         # special cases
-        self.assertEquals(glob_to_re(r'foo\\*'), r'foo\\\\[^/]*$')
-        self.assertEquals(glob_to_re(r'foo\\\*'), r'foo\\\\\\[^/]*$')
-        self.assertEquals(glob_to_re('foo????'), r'foo[^/][^/][^/][^/]$')
-        self.assertEquals(glob_to_re(r'foo\\??'), r'foo\\\\[^/][^/]$')
+        self.assertEquals(glob_to_re(r'foo\\*'), r'foo\\\\[^/]*\Z(?ms)')
+        self.assertEquals(glob_to_re(r'foo\\\*'), r'foo\\\\\\[^/]*\Z(?ms)')
+        self.assertEquals(glob_to_re('foo????'), r'foo[^/][^/][^/][^/]\Z(?ms)')
+        self.assertEquals(glob_to_re(r'foo\\??'), r'foo\\\\[^/][^/]\Z(?ms)')
 
 def test_suite():
     return unittest.makeSuite(FileListTestCase)