summaryrefslogtreecommitdiff
path: root/setuptools/config/expand.py
diff options
context:
space:
mode:
authorAnderson Bravalheri <andersonbravalheri@gmail.com>2022-04-04 20:42:07 +0100
committerAnderson Bravalheri <andersonbravalheri@gmail.com>2022-04-05 10:44:50 +0100
commit00b4fb1aef3feb77f9db0cd05bfeb02d1fa1cf75 (patch)
treee95c911d4daa00954abdbf124580b4963cb77159 /setuptools/config/expand.py
parentb686a319a2938019039c73aecba714970f9d6f74 (diff)
downloadpython-setuptools-git-00b4fb1aef3feb77f9db0cd05bfeb02d1fa1cf75.tar.gz
Simplify auto-discovered package_dir
If the directory follows a src-layout-ish, try harder to make `package_dir` in the form `{"": "src"}`. This might be later important for PEP 660 (e.g. when composing pth files or symlinking the toplevel packages).
Diffstat (limited to 'setuptools/config/expand.py')
-rw-r--r--setuptools/config/expand.py27
1 files changed, 25 insertions, 2 deletions
diff --git a/setuptools/config/expand.py b/setuptools/config/expand.py
index ff9b2c9b..156d7473 100644
--- a/setuptools/config/expand.py
+++ b/setuptools/config/expand.py
@@ -312,8 +312,12 @@ def find_packages(
where = kwargs.pop('where', ['.'])
packages: List[str] = []
fill_package_dir = {} if fill_package_dir is None else fill_package_dir
+ find = list(unique_everseen(always_iterable(where)))
- for path in unique_everseen(always_iterable(where)):
+ if len(find) == 1 and all(not _same_path(find[0], x) for x in (".", root_dir)):
+ fill_package_dir.setdefault("", find[0])
+
+ for path in find:
package_path = _nest_path(root_dir, path)
pkgs = PackageFinder.find(package_path, **kwargs)
packages.extend(pkgs)
@@ -326,8 +330,27 @@ def find_packages(
return packages
+def _same_path(p1: _Path, p2: _Path) -> bool:
+ """Differs from os.path.samefile because it does not require paths to exist.
+ Purely string based (no comparison between i-nodes).
+ >>> _same_path("a/b", "./a/b")
+ True
+ >>> _same_path("a/b", "a/./b")
+ True
+ >>> _same_path("a/b", "././a/b")
+ True
+ >>> _same_path("a/b", "./a/b/c/..")
+ True
+ >>> _same_path("a/b", "../a/b/c")
+ False
+ >>> _same_path("a", "a/b")
+ False
+ """
+ return os.path.normpath(p1) == os.path.normpath(p2)
+
+
def _nest_path(parent: _Path, path: _Path) -> str:
- path = parent if path == "." else os.path.join(parent, path)
+ path = parent if path in {".", ""} else os.path.join(parent, path)
return os.path.normpath(path)