Revert "Correctly support absolute paths in .dockerignore"

This reverts commit 34d5048.

Signed-off-by: mefyl <[email protected]>

Author: mefyl

docker/utils/build.py

@@ -46,7 +46,7 @@ def exclude_paths(root, patterns, dockerfile=None):
     )
 
 
-def should_include(path, exclude_patterns, include_patterns, root):
+def should_include(path, exclude_patterns, include_patterns):
     """
     Given a path, a list of exclude patterns, and a list of inclusion patterns:
 
@@ -61,15 +61,11 @@ def should_include(path, exclude_patterns, include_patterns, root):
             for pattern in include_patterns:
                 if match_path(path, pattern):
                     return True
-                if os.path.isabs(pattern) and match_path(
-                        os.path.join(root, path), pattern):
-                    return True
             return False
     return True
 
 
-def should_check_directory(directory_path, exclude_patterns, include_patterns,
-                           root):
+def should_check_directory(directory_path, exclude_patterns, include_patterns):
     """
     Given a directory path, a list of exclude patterns, and a list of inclusion
     patterns:
@@ -95,7 +91,7 @@ def normalize_path(path):
         if (pattern + '/').startswith(path_with_slash)
     ]
     directory_included = should_include(
-        directory_path, exclude_patterns, include_patterns, root
+        directory_path, exclude_patterns, include_patterns
     )
     return directory_included or len(possible_child_patterns) > 0
 
@@ -114,28 +110,26 @@ def get_paths(root, exclude_patterns, include_patterns, has_exceptions=False):
         # traversal. See https://docs.python.org/2/library/os.html#os.walk
         dirs[:] = [
             d for d in dirs if should_check_directory(
-                os.path.join(parent, d), exclude_patterns, include_patterns,
-                root
+                os.path.join(parent, d), exclude_patterns, include_patterns
             )
         ]
 
         for path in dirs:
             if should_include(os.path.join(parent, path),
-                              exclude_patterns, include_patterns, root):
+                              exclude_patterns, include_patterns):
                 paths.append(os.path.join(parent, path))
 
         for path in files:
             if should_include(os.path.join(parent, path),
-                              exclude_patterns, include_patterns, root):
+                              exclude_patterns, include_patterns):
                 paths.append(os.path.join(parent, path))
 
     return paths
 
 
 def match_path(path, pattern):
-
     pattern = pattern.rstrip('/' + os.path.sep)
-    if pattern and not os.path.isabs(pattern):
+    if pattern:
         pattern = os.path.relpath(pattern)
 
     pattern_components = pattern.split(os.path.sep)

tests/unit/utils_test.py

@@ -876,13 +876,6 @@ def test_trailing_double_wildcard(self):
             )
         )
 
-    def test_exclude_include_absolute_path(self):
-        base = make_tree([], ['a.py', 'b.py'])
-        assert exclude_paths(
-            base,
-            ['/*', '!' + os.path.join(base, '*.py')]
-        ) == set(['a.py', 'b.py'])
-
 
 class TarTest(unittest.TestCase):
     def test_tar_with_excludes(self):
@@ -1033,52 +1026,52 @@ class ShouldCheckDirectoryTest(unittest.TestCase):
 
     def test_should_check_directory_not_excluded(self):
         assert should_check_directory(
-            'not_excluded', self.exclude_patterns, self.include_patterns, '.'
+            'not_excluded', self.exclude_patterns, self.include_patterns
         )
         assert should_check_directory(
             convert_path('dir/with'), self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
 
     def test_shoud_check_parent_directories_of_excluded(self):
         assert should_check_directory(
-            'dir', self.exclude_patterns, self.include_patterns, '.'
+            'dir', self.exclude_patterns, self.include_patterns
         )
         assert should_check_directory(
             convert_path('dir/with'), self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
 
     def test_should_not_check_excluded_directories_with_no_exceptions(self):
         assert not should_check_directory(
             'exclude_rather_large_directory', self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
         assert not should_check_directory(
             convert_path('dir/with/subdir_excluded'), self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
 
     def test_should_check_excluded_directory_with_exceptions(self):
         assert should_check_directory(
             convert_path('dir/with/exceptions'), self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
         assert should_check_directory(
             convert_path('dir/with/exceptions/in'), self.exclude_patterns,
-            self.include_patterns, '.'
+            self.include_patterns
         )
 
     def test_should_not_check_siblings_of_exceptions(self):
         assert not should_check_directory(
             convert_path('dir/with/exceptions/but_not_here'),
-            self.exclude_patterns, self.include_patterns, '.'
+            self.exclude_patterns, self.include_patterns
         )
 
     def test_should_check_subdirectories_of_exceptions(self):
         assert should_check_directory(
             convert_path('dir/with/exceptions/like_this_one/subdir'),
-            self.exclude_patterns, self.include_patterns, '.'
+            self.exclude_patterns, self.include_patterns
         )